Skip to content

gh-157695: Check that C API macros start with "Py" #157696

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
vstinner merged 20 commits into from
Sep 21, 2026
Merged

gh-157695: Check that C API macros start with "Py" #157696

Changes from 1 commit
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
e48804c
gh-157695: Check that C API macros start with "Py"
vstinner Sep 17, 2026
0d6621d
Add a comment to the ignore file
vstinner Sep 17, 2026
714f1f5
Remove duplicates
vstinner Sep 17, 2026
44eb795
Fix error message
vstinner Sep 17, 2026
819558a
Remove NEWS entry
vstinner Sep 17, 2026
c6af62b
Parse also pyconfig.h.in
vstinner Sep 18, 2026
998dd34
Update Tools/build/check_capi_macros.py
vstinner Sep 18, 2026
fa62aac
Update .github/workflows/build.yml
vstinner Sep 18, 2026
6188980
Cleanup regexs
vstinner Sep 18, 2026
8c09a6c
Move the check fro Makefile to pre-commit
vstinner Sep 18, 2026
7c8c77d
Check also if the ignored list is outdated
vstinner Sep 18, 2026
80a2e2f
Parse also #undef in Include/ headers
vstinner Sep 18, 2026
d2347d9
Check that the ignore list is sorted
vstinner Sep 18, 2026
a63e018
Update Tools/build/check_capi_macros.py
vstinner Sep 19, 2026
86514dc
Accept spaces in more places in regex
vstinner Sep 19, 2026
dec9c88
Check for duplicated entires in the ignore list
vstinner Sep 19, 2026
d326d3f
Merge branch 'main' into check_capi_macros
vstinner Sep 19, 2026
d7a9559
Remove HAVE_FLOAT16 macro from ignore list
vstinner Sep 19, 2026
dd15660
Exclude 7 header files
vstinner Sep 20, 2026
4e78617
Combine "define" and "undef"; add line numbers (#19)
encukou Sep 21, 2026
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
Update Tools/build/check_capi_macros.py 
Co-authored-by: Hugo van Kemenade <1324225+hugovk@users.noreply.github.com>
  • Loading branch information
@vstinner @hugovk
vstinner and hugovk authored Sep 19, 2026
commit a63e01867891db2e06454afab6e63ff6064fef3b
2 changes: 1 addition & 1 deletion Tools/build/check_capi_macros.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -103,7 +103,7 @@ def main():
"the following macros can be removed:")
print()
for name in sorted(outdated):
print(f" - {name}")
print(f"- {name}")
print()
print(f"Total: {len(outdated)} macros")
print()
Expand Down
Loading