-
-
Notifications
You must be signed in to change notification settings - Fork 409
plugins: improve docstrings according to project standards #2002
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
Conversation
dgw
left a comment
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.
Guess I can't just put this as "Comment"; there are too many notes. "Request changes" it is!
dgw
left a comment
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.
This is probably ready to squash, since I'm down to nitpicking consistency of word choices. 😹
Co-authored-by: dgw <[email protected]>
3dfa8b0 to
aaa931b
Compare
|
Squash done. |
dgw
left a comment
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.
Let's ![]()
Description
Part of #1565 for
sopel.plugins.Most of the changes are missing
.and removing initial capital letter inparam/returntags.Then I tried to improve the docstrings for
sopel.plugins.handlers, in a somewhat better version.Checklist
make qa(runsmake qualityandmake test)