Skip to content

gh-133143: Condense the implementation for sys.abi_info#138672

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

Jump to bottom
Merged
AA-Turner merged 32 commits into from
Sep 8, 2025
Merged

gh-133143: Condense the implementation for sys.abi_info#138672

AA-Turner merged 32 commits into from
Sep 8, 2025

Conversation

@AA-Turner
Copy link
Member

@AA-TurnerAA-Turner commented Sep 8, 2025•
edited by bedevere-app bot
Loading

zklausand others added 30 commits September 3, 2025 23:47
@zklaus
Add sys.abi_info
2e3e54a
@blurb-it@zklaus
📜🤖 Added by blurb_it.
2ddafa4
@zklaus
Simplify pointer_bits handling
cdc50c3
@zklaus
Remove explicit C API function
6fb25a3
@zklaus
Make abi_info object a simple namespace with no explicit C API
4c3d83d
@zklaus
Cleanup scaffolding
820e002
@zklaus
Fix make_abi_info to be static
918521f
@zklaus
Add test
7863651
@zklaus
Fix test
81d75fc
@zklaus
Add link
41027a3
@zklaus
Rename attributes
56726ab
@zklaus@encukou
Add better attribute specification to docs
94b7f40
Co-authored-by: Petr Viktorin <[email protected]>
@zklaus
Base debug on Py_REF_DEBUG
6887d6c
@zklaus
Add byteorder
7ced683
@zklaus
Fix test
cc5ae8c
@zklaus
Improve error message in test
5c18587
@zklaus@vstinner
Apply suggestions from code review
d025d17
Co-authored-by: Victor Stinner <[email protected]>
@zklaus
Revert "Base debug on Py_REF_DEBUG"
a089371
This reverts commit 86f2a01.
@zklaus
Add whatsnew entry for improved sys module
44ce7c7
@zklaus@vstinner
Update Python/sysmodule.c
8f57f17
Co-authored-by: Victor Stinner <[email protected]>
@zklaus@AA-Turner
Integrate suggestions from Adam Turner
77f7b9b
Co-authored-by: Adam Turner <[email protected]>
@zklaus
Add Windows configuration equivalents
312bff1
@zklaus
Fix rst lint
ed7b6d9
@encukou
Use Sphinx markup for attributes
15f065a
@zklaus
Merge pull request#3from encukou/add-sys.abi_info
151eb4c
sys.abi_info: Use Sphinx markup for attributes
@AA-Turner
Merge branch 'main' into add-sys.abi_info
9f5ffae
@encukou@AA-Turner
Don't use ReST in a C comment
128b150
Co-authored-by: Adam Turner <[email protected]>
@AA-Turner
docs rephrasing
0506533
@AA-Turner
Condense the implementation
c0fcade
@AA-Turner
Merge branch 'main' into add-sys.abi_info
87f02ef
# Conflicts: # Lib/test/test_sys.py # Misc/NEWS.d/next/Core_and_Builtins/2025-08-06-16-55-44.gh-issue-133143.l7CI9v.rst # Python/sysmodule.c
@AA-TurnerAA-Turner changed the title Condense the implementation for sys.abi_infogh-133143: Condense the implementation for sys.abi_infoSep 8, 2025
@bedevere-appbedevere-appbot mentioned this pull request Sep 8, 2025
Closed
encukou
encukou reviewed Sep 8, 2025
View reviewed changes
encukou
encukou reviewed Sep 8, 2025
View reviewed changes
@StanFromIrelandStanFromIreland added the type-refactor Code refactoring (with no changes in behavior) label Sep 8, 2025
picnixz
picnixz approved these changes Sep 8, 2025
View reviewed changes
@AA-TurnerAA-Turner enabled auto-merge (squash) September 8, 2025 18:56
@AA-TurnerAA-Turner merged commit 5443b9e into python:mainSep 8, 2025
45 checks passed
lkollar pushed a commit to lkollar/cpython that referenced this pull request Sep 9, 2025
@AA-Turner@lkollar
pythongh-133143: Condense the implementation forsys.abi_info(py…
0a6513f
…thon#138672)
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@encukouencukouencukou left review comments

@picnixzpicnixzpicnixz approved these changes

@ericsnowcurrentlyericsnowcurrentlyAwaiting requested review from ericsnowcurrentlyericsnowcurrently is a code owner

Assignees

No one assigned

Labels

skip newstype-refactorCode refactoring (with no changes in behavior)

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@AA-Turner@encukou@picnixz@StanFromIreland@zklaus