Skip to content

gh-132805: annotationlib: Fix handling of non-constant values in FORWARDREF #132812

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.

Sign up for GitHub

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
JelleZijlstra merged 15 commits into from
May 4, 2025
Merged

gh-132805: annotationlib: Fix handling of non-constant values in FORWARDREF #132812

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
fba5efa
Add a test for union forward references
DavidCEllis Apr 22, 2025
d810953
Make stringifiers create unions if create_unions is True
DavidCEllis Apr 22, 2025
1908a4a
modify broken test, move test to forwardref format group
DavidCEllis Apr 22, 2025
97abdc7
Apparently trim trailing whitespace was turned off
DavidCEllis Apr 22, 2025
dd36a62
Alternative approach
JelleZijlstra Apr 22, 2025
20f3dac
Merge remote-tracking branch 'upstream/main' into fix-union-forwardref
JelleZijlstra Apr 22, 2025
8266a5e
fixes
JelleZijlstra Apr 22, 2025
1c629dc
blurb
JelleZijlstra Apr 22, 2025
3d7fec8
Handle displays
JelleZijlstra Apr 22, 2025
ec4dd0e
remove create_unions
JelleZijlstra Apr 22, 2025
6e31c69
prefer empty dict
JelleZijlstra Apr 22, 2025
525a4e4
Do not use special names in STRING format
JelleZijlstra Apr 23, 2025
0a75e5f
Merge branch 'main' into extra-names
JelleZijlstra May 4, 2025
7dec46b
Merge branch 'main' into extra-names
JelleZijlstra May 4, 2025
22a9691
fix bad merge
JelleZijlstra May 4, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'main' into extra-names
  • Loading branch information
@JelleZijlstra
JelleZijlstra authored May 4, 2025
commit 7dec46bbf394d7ab8d62992a29c40866da40e5bf
4 changes: 2 additions & 2 deletions Lib/annotationlib.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -708,8 +708,8 @@ def _stringify_single(anno):
return repr(anno)


def get_annotate_function(obj):
"""Get the __annotate__ function for an object.
def get_annotate_from_class_namespace(obj):
"""Retrieve the annotate function from a class namespace dictionary.

Return None if the namespace does not contain an annotate function.
This is useful in metaclass ``__new__`` methods to retrieve the annotate function.
Expand Down
Loading
You are viewing a condensed version of this merge commit. You can view the full changes here.