Skip to content

Conversation

JelleZijlstra
Copy link
Member

Fixes #91491

@bedevere-bot bedevere-bot added docs Documentation in the Doc dir awaiting core review labels May 12, 2022
@JelleZijlstra JelleZijlstra marked this pull request as draft May 12, 2022 03:07
Comment on lines +754 to +756
* Loosen runtime requirements for type annotations by removing the callable
check in the private ``typing._type_check`` function. (Contributed by
Gregory Beauregard in :gh:`90802`.)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we really include this? It's a private undocumented function (though David's recent bug report suggested to me otherwise).

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Given that it's already spawned a bug report, it's likely that other users will run into this change, so I think it's important to include.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed then. Could you please add a notice strongly discouraging users from using it too? We might as well try to dissuade users though it might be in vain :).

This function is for internal use only and is **unstable** and may change across versions. Please do not use this function. Instead, we recommend implementing this with your own code.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That feels like a bit much for the What's New document. We should say what changed, but I don't think we need more than that.

@JelleZijlstra JelleZijlstra marked this pull request as ready for review May 12, 2022 14:22
Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for working on this!

Co-authored-by: Alex Waygood <[email protected]>
@JelleZijlstra JelleZijlstra added the needs backport to 3.11 only security fixes label May 19, 2022
@JelleZijlstra JelleZijlstra merged commit 6b51773 into python:main May 19, 2022
@miss-islington
Copy link
Contributor

Thanks @JelleZijlstra for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

@miss-islington
Copy link
Contributor

Sorry @JelleZijlstra, I had trouble checking out the 3.11 backport branch.
Please backport using cherry_picker on command line.
cherry_picker 6b51773afd5658e15d23ce220f66fcc39c02dcae 3.11

@JelleZijlstra JelleZijlstra deleted the whatsnew branch May 19, 2022 14:07
@JelleZijlstra JelleZijlstra added needs backport to 3.11 only security fixes and removed needs backport to 3.11 only security fixes labels May 19, 2022
@miss-islington
Copy link
Contributor

Thanks @JelleZijlstra for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒⛏🤖

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request May 19, 2022
Fixes pythonGH-91491

Co-authored-by: Shantanu <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
(cherry picked from commit 6b51773)

Co-authored-by: Jelle Zijlstra <[email protected]>
@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label May 19, 2022
@bedevere-bot
Copy link

GH-92958 is a backport of this pull request to the 3.11 branch.

miss-islington added a commit that referenced this pull request May 19, 2022
Fixes GH-91491

Co-authored-by: Shantanu <[email protected]>
Co-authored-by: Alex Waygood <[email protected]>
(cherry picked from commit 6b51773)

Co-authored-by: Jelle Zijlstra <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
docs Documentation in the Doc dir skip news topic-typing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add typing to 3.11 What's New
6 participants