Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test: all public API methods should be documented with Javadoc #1867

Merged
merged 19 commits into from
Feb 21, 2018

Conversation

monperrus
Copy link
Collaborator

With this new architectural test, the CI will check that all public methods added in a PR are documented.

Note that the check is not yet fully complete, because the documentation debt was too high.
There remains small methods that are undocumented (methods with less than 35 expressions):

method.filterChildren(new TypeFilter<>(CtCodeElement.class)).list().size() > 35

upon acceptance, I will a good first issue to document the remaining methods.

@surli surli merged commit b4950d7 into INRIA:master Feb 21, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants