8000 [3.10] bpo-46581: Propagate private vars via _GenericAlias.copy_with (GH-31061) by posita · Pull Request #31821 · python/cpython · GitHub
[go: up one dir, main page]

Skip to content

[3.10] bpo-46581: Propagate private vars via _GenericAlias.copy_with (GH-31061) #31821

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

Merged
merged 1 commit into from
Mar 11, 2022

Conversation

posita
Copy link
Contributor
@posita posita commented Mar 11, 2022

(Cherry-picked from 32bf359.)

GH-26091 added the _typevar_types and _paramspec_tvars instance
variables to _GenericAlias. However, they were not propagated
consistently. This commit addresses the most prominent deficiency
identified in bpo-46581 (namely their absence from
_GenericAlias.copy_with), but there could be others.

Co-authored-by: Ken Jin 28750310+Fidget-Spinner@users.noreply.github.com
Co-authored-by: Serhiy Storchaka storchaka@gmail.com

…ythonGH-31061)

(Cherry-picked from 32bf359.)

pythonGH-26091 added the _typevar_types and _paramspec_tvars instance
variables to _GenericAlias. However, they were not propagated
consistently. This commit addresses the most prominent deficiency
identified in bpo-46581 (namely their absence from
_GenericAlias.copy_with), but there could be others.

Co-authored-by: Ken Jin <28750310+Fidget-Spinner@users.noreply.github.com>
Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@posita
Copy link
Contributor Author
posita commented Mar 11, 2022

Question: Does this fight with #31143? Is this going to inadvertently revert that approach? If so, is that okay?

@serhiy-storchaka
Copy link
Member

#31143 will not be backported. Also, it only reverts a part of changes.

@serhiy-storchaka serhiy-storchaka marked this pull request as ready for review March 11, 2022 14:57
@serhiy-storchaka serhiy-storchaka merged commit 3bc8019 into python:3.10 Mar 11, 2022
@posita posita deleted the posita/0/backport-32bf359-3.10 branch March 11, 2022 15:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants
0