-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
[Documentation] bot.py doc fixes #1884
Conversation
All docs obtained from official Bot API docs
There is also documentation related to API 4.7 in there |
Wow, thanks! Will have to merge after #1858 though, if there is API 4.7 docs in there.
If you install the pre-commit hooks, git will check flake8 before commiting ;) see our contrib-guide |
Yea I didn't add all docs related to 4.7 since the new methods aren't there in |
I guessed so ;) But we'll release the API 4.7 changes soon.
You only need a command line. Non-GUI stuff was scary to me at first, too, but it's worth it :) |
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 did a bit of nitpicking ;)
Please build the docs on your local machine to make sure that everything renders correctly. See the contrib-guide for details on that (you're gonna have to use the command line for that …)
Yea I finally set up this command line thing and installed the pre-commit hooks. I'll now use it to render the docs too. |
Also improved consistency of `returns:` in docs
CI fails unrelated (this is docs only). merging. |
All docs obtained from official Bot API docs.