Skip to content

gh-144629: Add test for the PyFunction_GetAnnotations function - #144630

Merged
vstinner merged 1 commit into
python:mainfrom
nybblista:test-pyfunc-anno
Feb 10, 2026
Merged

gh-144629: Add test for the PyFunction_GetAnnotations function#144630
vstinner merged 1 commit into
python:mainfrom
nybblista:test-pyfunc-anno

Conversation

@nybblista

@nybblistanybblista commented Feb 9, 2026•
edited by bedevere-app Bot
Loading

Copy link
Copy Markdown
Contributor

Add test for the PyFunction_GetAnnotations function.

@bedevere-appbedevere-appBot mentioned this pull request Feb 9, 2026
Closed
@nybblista

nybblista commented Feb 9, 2026

Copy link
Copy Markdown
ContributorAuthor

@vstinner

vstinner
vstinner reviewed Feb 9, 2026
View reviewed changes
Comment threadLib/test/test_capi/test_function.py Outdated
Comment threadModules/_testcapi/function.c Outdated
Comment threadModules/_testcapi/function.c Outdated
@nybblista

nybblista commented Feb 9, 2026

Copy link
Copy Markdown
ContributorAuthor

The other test functions in the function.c file use the same else style, so I kept it for consistency.

ZeroIntensity
ZeroIntensity reviewed Feb 10, 2026
View reviewed changes
Comment threadModules/_testcapi/function.c Outdated
@ZeroIntensityZeroIntensity added skip news needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes tests Tests in the Lib/test dir labels Feb 10, 2026
@nybblista

nybblista commented Feb 10, 2026

Copy link
Copy Markdown
ContributorAuthor

💯 I agree with @ZeroIntensity. However, I think it would be a good idea to apply the same approach to function_get_code, function_get_globals, and function_get_module.

@vstinner

vstinner commented Feb 10, 2026

Copy link
Copy Markdown
Member

However, I think it would be a good idea to apply the same approach to function_get_code, function_get_globals, and function_get_module.

I don't think that it's worth it to modify existing functions, it would only be a coding style change.

vstinner
vstinner approved these changes Feb 10, 2026
View reviewed changes

@vstinnervstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@vstinner
vstinner enabled auto-merge (squash) February 10, 2026 14:11
@vstinner
vstinner merged commit cc81707 into python:mainFeb 10, 2026
85 of 87 checks passed
@miss-islington-app

miss-islington-appBot commented Feb 10, 2026

Copy link
Copy Markdown

Thanks @nybblista for the PR, and @vstinner for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13, 3.14.
🐍🍒⛏🤖 I'm not a witch! I'm not a witch!

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Feb 10, 2026
@nybblista@miss-islington
pythongh-144629: Add test for the PyFunction_GetAnnotations() function (
b75577b
pythonGH-144630)
(cherry picked from commit cc81707)
Co-authored-by: Nybblista <170842536+nybblista@users.noreply.github.com>
@miss-islington-app

miss-islington-appBot commented Feb 10, 2026

Copy link
Copy Markdown

Sorry, @nybblista and @vstinner, I could not cleanly backport this to 3.13 due to a conflict.
Please backport using cherry_picker on command line.

cherry_picker cc81707e406c49c63afc18048e1a221d796ce638 3.13

@bedevere-app

bedevere-appBot commented Feb 10, 2026

Copy link
Copy Markdown

GH-144670 is a backport of this pull request to the 3.14 branch.

@bedevere-appbedevere-appBot removed the needs backport to 3.14 bugs and security fixes label Feb 10, 2026
@vstinnervstinner removed the needs backport to 3.13 bugs and security fixes label Feb 10, 2026
@vstinner

vstinner commented Feb 10, 2026

Copy link
Copy Markdown
Member

vstinner removed the needs backport to 3.13 label

3.13 branch lacks Modules/_testcapi/function.c file. I'm not interested to backport it.

@vstinner

vstinner commented Feb 10, 2026

Copy link
Copy Markdown
Member

Merged, thanks.

@nybblista
nybblista deleted the test-pyfunc-anno branch February 10, 2026 14:50
vstinner pushed a commit that referenced this pull request Feb 10, 2026
@miss-islington@nybblista
[3.14]gh-144629: Add test for the PyFunction_GetAnnotations() functi…
befa954
…on (GH-144630) (#144670)
gh-144629: Add test for the PyFunction_GetAnnotations() function (GH-144630)
(cherry picked from commit cc81707)
Co-authored-by: Nybblista <170842536+nybblista@users.noreply.github.com>
thunder-coding pushed a commit to thunder-coding/cpython that referenced this pull request Feb 15, 2026
@nybblista@thunder-coding
pythongh-144629: Add test for the PyFunction_GetAnnotations() function (
436181f
python#144630)
brijkapadia pushed a commit to brijkapadia/cpython that referenced this pull request Feb 28, 2026
@nybblista@brijkapadia
pythongh-144629: Add test for the PyFunction_GetAnnotations() function (
4601e9c
python#144630)
ljfp pushed a commit to ljfp/cpython that referenced this pull request Apr 25, 2026
@nybblista@ljfp
pythongh-144629: Add test for the PyFunction_GetAnnotations() function (
b18331d
python#144630)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinnervstinnervstinner approved these changes
@ZeroIntensityZeroIntensityAwaiting requested review from ZeroIntensity

Assignees

@vstinnervstinner

Labels

skip newstestsTests in the Lib/test dir

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@nybblista@vstinner@ZeroIntensity