-
-
Notifications
You must be signed in to change notification settings - Fork 33k
GH-138465: Improve documentation for common sequence methods #138474
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-138465: Improve documentation for common sequence methods #138474
Conversation
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.
Looks great!
Co-authored-by: Serhiy Storchaka <[email protected]>
Mutable sequences should provide methods :meth:`~sequence.append`, | ||
:meth:`~sequence.count`, :meth:`~sequence.index`, :meth:`~sequence.extend`, | ||
:meth:`~sequence.insert`, :meth:`~sequence.pop`, :meth:`~sequence.remove`, | ||
:meth:`~sequence.reverse` and :meth:`~sequence.sort`, |
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.
sequence.sort
does not exist. But sequence.clear
was omitted.
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 unchanged text (the only difference is fixed cross-references). Please could we leave it to a different issue to change, if needed?
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.
Opened #138562
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.
LGTM. 👍
Thanks @AA-Turner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
…ythonGH-138474) (cherry picked from commit 8ed1d53) Co-authored-by: Adam Turner <[email protected]> Co-authored-by: Serhiy Storchaka <[email protected]>
Sorry, @AA-Turner, I could not cleanly backport this to
|
GH-138560 is a backport of this pull request to the 3.14 branch. |
…hods (pythonGH-138474) (cherry picked from commit 8ed1d53) Co-authored-by: Adam Turner <[email protected]> Co-authored-by: Serhiy Storchaka <[email protected]>
GH-138567 is a backport of this pull request to the 3.13 branch. |
…H-138474) (#138567) Co-authored-by: Serhiy Storchaka <[email protected]>
…H-138474) (#138560) GH-138465: Improve documentation for common sequence methods (GH-138474) (cherry picked from commit 8ed1d53) Co-authored-by: Adam Turner <[email protected]> Co-authored-by: Serhiy Storchaka <[email protected]>
…hods (pythonGH-138474) (python#138560) pythonGH-138465: Improve documentation for common sequence methods (pythonGH-138474) (cherry picked from commit 8ed1d53) Co-authored-by: Adam Turner <[email protected]> Co-authored-by: Serhiy Storchaka <[email protected]>
…ython#138474) Co-authored-by: Serhiy Storchaka <[email protected]>
cc @serhiy-storchaka
As part of this I've gone through the rest of the documentation and updated cross-references to use
sequence.{method}
/list.{method}
etc as appropriate.A
📚 Documentation preview 📚: https://cpython-previews--138474.org.readthedocs.build/