Skip to content

[3.14] gh-137226: Fix behavior of ForwardRef.evaluate with type_params (GH-137227)#137709

Merged
hugovk merged 1 commit into
python:3.14from
miss-islington:backport-089a324-3.14
Aug 13, 2025
Merged

[3.14] gh-137226: Fix behavior of ForwardRef.evaluate with type_params (GH-137227)#137709
hugovk merged 1 commit into
python:3.14from
miss-islington:backport-089a324-3.14

Conversation

@miss-islington

@miss-islington miss-islington commented Aug 13, 2025

Copy link
Copy Markdown
Contributor

The previous behavior was copied from earlier typing code. It works around the way
typing.get_type_hints passes its namespaces, but I don't think the behavior is logical
or correct.
(cherry picked from commit 089a324)

Co-authored-by: Jelle Zijlstra jelle.zijlstra@gmail.com

…pythonGH-137227)

The previous behavior was copied from earlier typing code. It works around the way
typing.get_type_hints passes its namespaces, but I don't think the behavior is logical
or correct.
(cherry picked from commit 089a324)

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
@JelleZijlstra

Copy link
Copy Markdown
Member

@hugovk is it still possible to get this in?

@ncoghlan ncoghlan left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@hugovk

hugovk commented Aug 13, 2025

Copy link
Copy Markdown
Member

@hugovk is it still possible to get this in?

Yep!

@hugovk hugovk enabled auto-merge (squash) August 13, 2025 13:54
@hugovk hugovk merged commit 902de28 into python:3.14 Aug 13, 2025
51 checks passed
kumaraditya303 pushed a commit to miss-islington/cpython that referenced this pull request Sep 9, 2025
…_params (pythonGH-137227) (python#137709)

Co-authored-by: Jelle Zijlstra <jelle.zijlstra@gmail.com>
@miss-islington miss-islington deleted the backport-089a324-3.14 branch January 2, 2026 17:39
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