-
-
Notifications
You must be signed in to change notification settings - Fork 33.4k
gh-141004: Document pyctype.h macros
#141272
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
ZeroIntensity
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.
These seem like a candidate for soft deprecation, so please bring this up at capi-workgroup/decisions#86. I'm a bit surprised that they were intentionally added to the public API in 2.7, but character functions don't seem like CPython's job to support.
|
I concur, soft deprecation makes sense. |
ZeroIntensity
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.
Thanks for dealing with these! I'm going to hold off on merging until Monday in case someone else has a strong opinion on these.
|
@malemburg maybe you would like to review? |
|
Thanks @StanFromIreland for the PR, and @ZeroIntensity for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13. |
|
Thanks @StanFromIreland for the PR, and @ZeroIntensity for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14. |
(cherry picked from commit df19261) Co-authored-by: Stan Ulbrych <[email protected]>
|
GH-141339 is a backport of this pull request to the 3.13 branch. |
(cherry picked from commit df19261) Co-authored-by: Stan Ulbrych <[email protected]>
|
GH-141340 is a backport of this pull request to the 3.14 branch. |
📚 Documentation preview 📚: https://cpython-previews--141272.org.readthedocs.build/