-
-
Notifications
You must be signed in to change notification settings - Fork 31k
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-105071: add PyUnstable_Exc_PrepReraiseStar to expose except* implementation in the unstable API #105072
gh-105071: add PyUnstable_Exc_PrepReraiseStar to expose except* implementation in the unstable API #105072
Conversation
… implementation in the unstable API
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.
Doc formatting nits. Otherwise: lgtm, as far as I can see.
Co-authored-by: Erlend E. Aasland <erlend.aasland@protonmail.com>
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 good, thanks!
🤖 New build scheduled with the buildbot fleet by @iritkatriel for commit 784b102 🤖 If you want to schedule another build, you need to add the 🔨 test-with-refleak-buildbots label again. |
Thanks @iritkatriel for the PR 🌮🎉.. I'm working now to backport this PR to: 3.12. |
Sorry @iritkatriel, I had trouble checking out the |
…except* implementation in the unstable API (pythonGH-105072) (cherry picked from commit b7aadb4) Co-authored-by: Irit Katriel <1055913+iritkatriel@users.noreply.github.com>
GH-105095 is a backport of this pull request to the 3.12 branch. |
GH-105095 is a backport of this pull request to the 3.12 branch. |
I did not rename the internal function, but rather added an API function that calls it. This is because the internal function can make more assumptions about the input than an API function (so the API function raises exceptions on things that the internal function just asserts).
Fixes #105071.
CC @da-woods .
📚 Documentation preview 📚: https://cpython-previews--105072.org.readthedocs.build/