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

Percentage computation rework and more fixes #11

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

stieglma
Copy link

  • override annotations are now working for supertyes and interfaces (only supertypes before)
  • all interesting numbers are added to the report (some were left out before, e.g. enum constants)
  • changed htmlexporter a little bit to have a more logical structure
  • enum methods values and valueOf were always shown as undocumented although they are predefined and can't even be overridden by the user

Feel free to add comments to the pull request where applicable. I was doing these changes in order to have a better view if my private/publicOnly changes were giving meaningful results.

Thomas Stieglmaier added 5 commits January 18, 2018 08:36
…ding:

- override annotations are now working for supertyes and interfaces (only supertypes before)
- all interesting numbers are added to the report (some were left out before, e.g. enum constants)
- changed htmlexporter a little bit to have a more logical structure
@stieglma
Copy link
Author

Actually I thought a bit more about the logic for counting overridden methods and I think it is still not completely correct: when the super class/interface is not documented, the overriding method is still counted as fully documented. I will add a fix for that on this branch ,too

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.

None yet

1 participant