Unifying the docs for the Poll class.#3312
Unifying the docs for the Poll class.#3312harshil21 merged 11 commits intopython-telegram-bot:doc-fixesfrom RobiMez:doc-fixes
Conversation
harshil21
left a comment
There was a problem hiding this comment.
Thanks for the PR, left some comments below
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
Co-authored-by: Harshil <37377066+harshil21@users.noreply.github.com>
|
pre-commit is failing since the line is too long (L129 & 154 in _poll.py). It should be < 100 chars |
|
ill add some breaks |
|
done , also a few questions , was the line feed bit unnecessary ? is it because its a niche thing that people don't run into or it just didnt belong there ? |
|
oh wait , trailing whitespace |
|
@harshil21 Ohhh , makes sense , getting the hang of sphinx and your linting checks too. Ill tackle another class tomorrow as its getting late here. any priorities / which one needs most work ? |
no priorities as of now, feel free to dive in the code and pick any class(es) you find which has a large discrepancy. Try to do multiple classes together so as to not produce many PRs for the same thing. |
Checklist for PRs
.. versionadded:: version,.. versionchanged:: versionor.. deprecated:: versionto the docstrings for user facing changes (for methods/class descriptions, arguments and attributes)AUTHORS.rst(optional)__all__s