Skip to content

Adjust stable ABI internal documentation #96896

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

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!

@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
@miss-islington
Copy link
Contributor

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

@bedevere-bot bedevere-bot removed the needs backport to 3.11 only security fixes label Oct 4, 2022
@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 <[email protected]>
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 <[email protected]>
@woodruffw woodruffw deleted the fix-stable-abi-docs branch October 4, 2022 16:49
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 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants