-
-
Notifications
You must be signed in to change notification settings - Fork 33k
gh-101100: Fix some Sphinx reference warnings in whatsnew/2.6.rst #139236
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
gh-101100: Fix some Sphinx reference warnings in whatsnew/2.6.rst #139236
Conversation
Also add a working link for abc.abstractmethod while there
@rowanbudge Looks like the CLA bot still needs appeasing. Please can you check it's signed with the same email address mentioned in the comment? |
I think this is done now, not sure when the github accounts got switched sorry! |
Doc/whatsnew/2.6.rst
Outdated
and :meth:`close`. | ||
* :class:`~io.RawIOBase` defines raw I/O operations: :meth:`~io.RawIOBase.read`, | ||
:meth:`~io.RawIOBase.readinto`, | ||
:meth:`~io.RawIOBase.write`, :meth:`~io.IOBase.seek`, :meth:`~io.IOBase.tell`, :meth:`~io.IOBase.truncate`, |
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.
Please wrap lines to 79 characters.
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've wrapped at 79 lines in all the places already edited by this PR that I could spot
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.
Thanks, however, it introduced a lot of trailing white space causing the lint CI to fail.
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.
Yep aware and working on 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.
Checks now pass
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.
On the other hand, rewrapping all edits makes it harder to review the actual changes in the diff, so it's a balance. (There's some discussion in python/devguide#1018.)
No need to unwrap though.
Also please avoid force-pushing -- everything gets squashed into a single commit at the end, and separate commits are easier to review. Thanks!
de1a24b
to
63b5bd2
Compare
now with pre-commit check to remove whitespace
b57ba9e
to
3c94412
Compare
Hopefully also resolve Block quote ends without a blank line; unexpected unindent.
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.
Thank you!
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.
Is this PR intended to resolve all warnings in the file? It's OK if not, just trying to get a sense of intended scope for the review.
A
No, not all, just one sprint day's worth :) We can fix the rest in future PRs. |
Thanks @rowanbudge for the PR, and @AA-Turner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
….rst`` (pythonGH-139236) (cherry picked from commit 04a2f80) Co-authored-by: rowanbudge <[email protected]> Co-authored-by: rowanvil <[email protected]> Co-authored-by: Adam Turner <[email protected]>
….rst`` (pythonGH-139236) (cherry picked from commit 04a2f80) Co-authored-by: rowanbudge <[email protected]> Co-authored-by: rowanvil <[email protected]> Co-authored-by: Adam Turner <[email protected]>
GH-139568 is a backport of this pull request to the 3.14 branch. |
GH-139569 is a backport of this pull request to the 3.13 branch. |
Thanks for this PR @rowanbudge and congratulations on your first contribution! A |
…6.rst`` (python#139236) Co-authored-by: rowanvil <[email protected]> Co-authored-by: Adam Turner <[email protected]>
For gh-101100 - Fix all Sphinx reference warnings in the documentation #101100
Fix some of the issues in whatsnew/2.6.rst
Part of sprints at PyConUK 25
📚 Documentation preview 📚: https://cpython-previews--139236.org.readthedocs.build/