Skip to content
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-94309: Deprecate typing.Hashable/Sized #94626

Merged

Conversation

MonadChains
Copy link
Contributor

@MonadChains MonadChains commented Jul 6, 2022

#94309
To be consistent with PEP 585, deprecated aliases should not raise any DeprecationWarning.

I put as first version of deprecation the upcoming Python 3.12 (I am not fully sure about this though).

MonadChains added 2 commits Jul 6, 2022
To be consistent with PEP 585, deprecated aliases should not raise any DeprecationWarning.
@cpython-cla-bot
Copy link

@cpython-cla-bot cpython-cla-bot bot commented Jul 6, 2022

All commit authors signed the Contributor License Agreement.
CLA signed

@bedevere-bot
Copy link

@bedevere-bot bedevere-bot commented Jul 6, 2022

Most changes to Python require a NEWS entry.

Please add it using the blurb_it web app or the blurb command-line tool.

@bedevere-bot bedevere-bot added the docs label Jul 6, 2022
@MonadChains MonadChains changed the title Gh 94309/deprecate hashed sized gh-94309: Deprecate typing.Hashable/Sized Jul 6, 2022
Copy link
Contributor

@Fidget-Spinner Fidget-Spinner left a comment

Deprecating in 3.12 is correct. I think this is almost ready to merge. Good work!

Doc/library/typing.rst Outdated Show resolved Hide resolved
@AlexWaygood AlexWaygood added expert-typing type-feature labels Jul 7, 2022
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Could you add a row to this table at the bottom of the page, as well? https://docs.python.org/3.12/library/typing.html#deprecation-timeline-of-major-features

Copy link
Contributor

@Fidget-Spinner Fidget-Spinner left a comment

LGTM.

Copy link
Member

@AlexWaygood AlexWaygood left a comment

LGTM too!

@Fidget-Spinner Fidget-Spinner merged commit e8e4b55 into python:main Jul 10, 2022
14 checks passed
@Fidget-Spinner
Copy link
Contributor

@Fidget-Spinner Fidget-Spinner commented Jul 10, 2022

@MonadChains congrats on your first contribution to CPython 🎉 . I look forward to more of your contributions in the future.

@hugovk
Copy link
Member

@hugovk hugovk commented Jul 10, 2022

Shall we also add this to "What’s New In Python 3.12" in the docs?

Because we don't have a set version for removal, it can go in the "Pending Removal in Future Versions" section:

https://docs.python.org/3.12/whatsnew/3.12.html#pending-removal-in-future-versions

(Sorry for the late suggestion!)

@Fidget-Spinner
Copy link
Contributor

@Fidget-Spinner Fidget-Spinner commented Jul 10, 2022

Shall we also add this to "What’s New In Python 3.12" in the docs?

Yes we should.

@MonadChains MonadChains deleted the gh-94309/deprecate-hashed-sized branch Jul 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs expert-typing type-feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

7 participants