-
Notifications
You must be signed in to change notification settings - Fork 45
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
Allow to choose members order of classes and modules #653
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #653 +/- ##
==========================================
+ Coverage 92.68% 92.70% +0.02%
==========================================
Files 47 47
Lines 8279 8304 +25
Branches 1828 1833 +5
==========================================
+ Hits 7673 7698 +25
Misses 347 347
Partials 259 259 ☔ View full report in Codecov by Sentry. |
…sentation order of members. Add some tests.
This looks good, I’m just unsure we absolutely need to introduce two new arguments… this sounds like a lot. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
…es from docstring fields. Should we store the linenumber as the docstring_lineno as well ?
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
1 similar comment
This comment has been minimized.
This comment has been minimized.
…d warnings at the correct location.
…or into 485-and-580-members-order
According to pydoctor_primer, this change doesn't affect pydoctor warnings on a corpus of open source code. ✅ |
Add options --cls-member-order and --mod-member-order, fixes #580 and fixes #485.