-
-
Notifications
You must be signed in to change notification settings - Fork 33.6k
GH-138562: Remove sort() from the common sequence methods in the data model
#138563
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
serhiy-storchaka
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.
Should not we add sequence.clear instead?
serhiy-storchaka
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.
LGTM. 👍
# Conflicts: # Doc/reference/datamodel.rst
|
Thanks @AA-Turner for the PR 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14. |
…n the data model (pythonGH-138563) (cherry picked from commit d0c9943) Co-authored-by: Adam Turner <[email protected]>
|
Sorry, @AA-Turner, I could not cleanly backport this to |
|
GH-138760 is a backport of this pull request to the 3.14 branch. |
…in the data model (GH-138563) (#138760) Co-authored-by: Adam Turner <[email protected]>
…thods in the data model (pythonGH-138563) (cherry picked from commit d0c9943) Co-authored-by: Adam Turner <[email protected]>
|
GH-140676 is a backport of this pull request to the 3.13 branch. |
…in the data model (GH-138563) (#140676) (cherry picked from commit d0c9943) Co-authored-by: Adam Turner <[email protected]>
sort()#138562📚 Documentation preview 📚: https://cpython-previews--138563.org.readthedocs.build/