Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

gh-105237: Allow calling issubclass(X, typing.Protocol) again #105239

Merged
merged 6 commits into from
Jun 5, 2023
Merged

gh-105237: Allow calling issubclass(X, typing.Protocol) again #105239

merged 6 commits into from
Jun 5, 2023

Conversation

AlexWaygood
Copy link
Member

@AlexWaygood AlexWaygood commented Jun 2, 2023 •
edited
Loading

@AlexWaygood AlexWaygood added type-bug An unexpected behavior, bug, or error skip news stdlib Python modules in the Lib dir topic-typing needs backport to 3.12 bug and security fixes labels Jun 2, 2023
@bedevere-bot bedevere-bot mentioned this pull request Jun 2, 2023
Closed
AlexWaygood
AlexWaygood commented Jun 2, 2023
View reviewed changes
Lib/test/test_typing.py Outdated Show resolved Hide resolved
@AlexWaygood AlexWaygood changed the title gh-105237: Allow calling issubclass(X, Protocol) again gh-105237: Allow calling issubclass(X, typing.Protocol) again Jun 2, 2023
JelleZijlstra
JelleZijlstra reviewed Jun 2, 2023
View reviewed changes
Lib/typing.py Outdated Show resolved Hide resolved
@AlexWaygood AlexWaygood requested a review from JelleZijlstra June 2, 2023 14:09
@AlexWaygood
Copy link
Member Author

I've run the third-party tests we have at typing_extensions on a version of this PR. In combination with #105281, this causes no regressions.

@AlexWaygood AlexWaygood mentioned this pull request Jun 4, 2023
Merged
@JelleZijlstra JelleZijlstra merged commit cdfb201 into python:main Jun 5, 2023
@miss-islington
Copy link
Contributor

Thanks @AlexWaygood for the PR, and @JelleZijlstra for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @AlexWaygood and @JelleZijlstra, I had trouble checking out the 3.12 backport branch.
Please retry by removing and re-adding the "needs backport to 3.12" label.
Alternatively, you can backport using cherry_picker on the command line.
cherry_picker cdfb201bfa35b7c50de5099c6d9078c806851d98 3.12

@JelleZijlstra JelleZijlstra added needs backport to 3.12 bug and security fixes and removed needs backport to 3.12 bug and security fixes labels Jun 5, 2023
@miss-islington
Copy link
Contributor

Thanks @AlexWaygood for the PR, and @JelleZijlstra for merging it 🌮🎉.. I'm working now to backport this PR to: 3.12.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Jun 5, 2023
@AlexWaygood @miss-islington
pythongh-105237: Allow calling issubclass(X, typing.Protocol) again (…
f45b5d7 
…pythonGH-105239)

(cherry picked from commit cdfb201)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@bedevere-bot
Copy link

GH-105316 is a backport of this pull request to the 3.12 branch.

@bedevere-bot bedevere-bot removed the needs backport to 3.12 bug and security fixes label Jun 5, 2023
@AlexWaygood AlexWaygood deleted the protocol-subclass-regression branch June 5, 2023 13:37
AlexWaygood added a commit that referenced this pull request Jun 5, 2023
@miss-islington @AlexWaygood
[3.12] gh-105237: Allow calling issubclass(X, typing.Protocol) again (
5175026 
GH-105239) (#105316)

gh-105237: Allow calling `issubclass(X, typing.Protocol)` again (GH-105239)
(cherry picked from commit cdfb201)

Co-authored-by: Alex Waygood <Alex.Waygood@Gmail.com>
@erlend-aasland erlend-aasland mentioned this pull request Jun 6, 2023
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@JelleZijlstra JelleZijlstra JelleZijlstra approved these changes

@gvanrossum gvanrossum Awaiting requested review from gvanrossum

@Fidget-Spinner Fidget-Spinner Awaiting requested review from Fidget-Spinner

Assignees

@JelleZijlstra JelleZijlstra

Labels
stdlib Python modules in the Lib dir topic-typing type-bug An unexpected behavior, bug, or error
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@AlexWaygood @miss-islington @bedevere-bot @JelleZijlstra