Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Backport NamedTuple and TypedDict deprecations from Python 3.13 #240
Backport NamedTuple and TypedDict deprecations from Python 3.13 #240
Changes from 5 commits
1bc23d0
51fc6a7
0092208
4f2b790
f4adac5
1cc1897
3354680
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
I took this opportunity to copy-paste the docstring from CPython.
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.
If we're adding a docstring the normal way here, we no longer need to monkey-patch it onto the function afterwards:
typing_extensions/src/typing_extensions.py
Line 2726 in f4adac5
If you get rid of the docstring monkey-patching, you'll need to ditch this test, as it'll start to fail:
typing_extensions/src/test_typing_extensions.py
Lines 5098 to 5100 in f9b83a2
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.
Oh I missed that. I think it's better if we present the same docstring across all versions, so I'll remove the monkey-patching.