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
Updates to satisfy pep257 violations #2028
base: main
Are you sure you want to change the base?
Conversation
There are lots more violations, Just posting this so folks can comment or help. |
7f3e7fa
to
257f105
Compare
This comment was marked as resolved.
This comment was marked as resolved.
426edaa
to
c7f0ce6
Compare
@radez wow, that's a lot of work! I'll have to dedicate some time to reviewing it more carefully. Meanwhile, since you seem to have addressed all of the rule violations, could you look into removing the exclusions? There's a pattern for temporarily excluded files in the pydocstyle entry of The next step could be replacing a separate |
Did that in 17b7f6d. |
@@ -54,33 +53,36 @@ class VirtualHost(object): | |||
|
|||
cherrypy.tree.graft(vhost) | |||
""" | |||
default = None | |||
"""Required. |
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.
I think it'd still be nice to have actual class attribute docstrings, not just initializer args.
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.
I don't disagree with you, there was just enough changes to make that I didn't put the time into having actual attribute docstrings
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.
Yeah.. It might've been easier to merge smaller PRs. I got to review a part of this yesterday, during the sprints, patching it file-by-file. But didn't get through all the files — people around needed help with other stuff they were sprinting on. I was hoping to keep going through it today.
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
Co-authored-by: Sviatoslav Sydorenko (Святослав Сидоренко) <wk.cvs.github@sydorenko.org.ua>
What kind of change does this PR introduce?
CI has a pep257 check in it that is riddled with violations.
This patch is an attempt to clean those up to help with enforcing new PRs pep257 compliance.