-
-
Notifications
You must be signed in to change notification settings - Fork 32k
gh-121025: improve partialmethod.__repr__
#121033
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-121025: improve partialmethod.__repr__
#121033
Conversation
I’m sorry, I am on vacation and dealing with health issues, and unavailable for review. |
(I'm also very sorry that you got requested for a single change in a regex) |
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 @picnixz for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13. |
Thanks @picnixz for the PR, and @serhiy-storchaka for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12. |
It no longer contains redundant commas and spaces. (cherry picked from commit d2646e3) Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
GH-121037 is a backport of this pull request to the 3.13 branch. |
It no longer contains redundant commas and spaces. (cherry picked from commit d2646e3) Co-authored-by: Bénédikt Tran <10796600+picnixz@users.noreply.github.com>
GH-121038 is a backport of this pull request to the 3.12 branch. |
It no longer contains redundant commas and spaces.
It no longer contains redundant commas and spaces.
It no longer contains redundant commas and spaces.
FYI,
partial.__repr__
is marked withrecursive_repr()
, but not partialmethod. For now, I did not mark it but if you think it should, I can add it.