X Tutup
The Wayback Machine - https://web.archive.org/web/20230120181332/https://github.com/python/cpython/pull/96896
Skip to content
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.

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

Adjust stable ABI internal documentation #96896

Merged
merged 6 commits into from Oct 4, 2022

Conversation

woodruffw
Copy link
Contributor

@woodruffw woodruffw commented Sep 17, 2022

I was perusing this file, and noticed that this part of the documentation is slightly out of date: the struct items in this TOML file currently contain struct_abi_kind members, which distinguish between the different types of ABI compatibility described in the comment.

I've updated the comment to reflect this.

Automerge-Triggered-By: GH:encukou

@woodruffw
Copy link
Contributor Author

(This is just a documentation tweak to an internal file, so I've skipped filing an issue. But let me know if you'd like me to.)

@encukou
Copy link
Member

encukou commented Sep 26, 2022

Thank you!
I see I haven't documented struct_abi_kind and members either. I filed woodruffw-forks#16, does it make sense to merge it into this PR?

@woodruffw
Copy link
Contributor Author

I see I haven't documented struct_abi_kind and members either. I filed woodruffw-forks#16, does it make sense to merge it into this PR?

I think so! Thanks for doing that!

encukou
encukou approved these changes Oct 4, 2022
@encukou encukou added 🤖 automerge PR will be merged once it's been approved and all CI passed needs backport to 3.11 labels Oct 4, 2022
@miss-islington
Copy link
Contributor

Sorry, I can't merge this PR. Reason: Pull Request is not mergeable.

@miss-islington miss-islington merged commit 6e53308 into python:main Oct 4, 2022
14 checks passed
@miss-islington
Copy link
Contributor

Thanks @woodruffw for the PR 🌮🎉.. I'm working now to backport this PR to: 3.11.
🐍🍒🤖

@bedevere-bot
Copy link

GH-97828 is a backport of this pull request to the 3.11 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Oct 4, 2022
I was perusing this file, and noticed that this part of the documentation is slightly out of date: the `struct` items in this TOML file currently contain `struct_abi_kind` members, which distinguish between the different types of ABI compatibility described in the comment.

I've updated the comment to reflect this.
(cherry picked from commit 6e53308)

Co-authored-by: William Woodruff <william@yossarian.net>
miss-islington added a commit that referenced this pull request Oct 4, 2022
I was perusing this file, and noticed that this part of the documentation is slightly out of date: the `struct` items in this TOML file currently contain `struct_abi_kind` members, which distinguish between the different types of ABI compatibility described in the comment.

I've updated the comment to reflect this.
(cherry picked from commit 6e53308)

Co-authored-by: William Woodruff <william@yossarian.net>
@woodruffw woodruffw deleted the fix-stable-abi-docs branch Oct 4, 2022
pablogsal pushed a commit that referenced this pull request Oct 22, 2022
I was perusing this file, and noticed that this part of the documentation is slightly out of date: the `struct` items in this TOML file currently contain `struct_abi_kind` members, which distinguish between the different types of ABI compatibility described in the comment.

I've updated the comment to reflect this.
(cherry picked from commit 6e53308)

Co-authored-by: William Woodruff <william@yossarian.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🤖 automerge PR will be merged once it's been approved and all CI passed skip issue skip news
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants
X Tutup