-
Notifications
You must be signed in to change notification settings - Fork 871
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
Enable more Pydocstyle rules #11582
Enable more Pydocstyle rules #11582
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Happy to defer enforcing the remaining rules to a future PR.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome ! Way less noise(warnings) during docs build 🥳 🕺
Codecov Report
@@ Coverage Diff @@
## branch-22.10 #11582 +/- ##
===============================================
Coverage ? 86.40%
===============================================
Files ? 145
Lines ? 22959
Branches ? 0
===============================================
Hits ? 19838
Misses ? 3121
Partials ? 0 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
Fantastic point! I hadn't thought to check that. |
@gpucibot merge |
Description
Recently I have reviewed a handful of PRs with problems in their docstrings that I've been fixing with GitHub review suggestions. I took 40 minutes and enabled a bunch of pydocstyle rules that we agreed on in #10711, to help prevent some of these problems and reduce the amount of review effort required for the future. There are a handful of big ones (
D200, D202, D205, D400
) that will require a more intense effort to implement -- those rules may not be worth the significant refactoring effort. I think this may resolve the part of #10711 that we wanted to tackle in the short term, though I'm happy to hear others' views (@shwina @vyasr).Error code reference: https://www.pydocstyle.org/en/stable/error_codes.html
Checklist