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

Jump to bottom

gh-101524: Only Use Public C-API in the _xxsubinterpreters Module #105258

Merged
merged 4 commits into from Jun 2, 2023
Merged

gh-101524: Only Use Public C-API in the _xxsubinterpreters Module #105258

merged 4 commits into from Jun 2, 2023

Conversation

ericsnowcurrently
Copy link
Member

@ericsnowcurrently ericsnowcurrently commented Jun 2, 2023 •
edited by bedevere-bot

The _xxsubinterpreters module was meant to only use public API. Some internal C-API usage snuck in over the last few years (e.g. gh-28969). This fixes that.

@ericsnowcurrently ericsnowcurrently requested a review from a team as a code owner June 2, 2023 22:28
@bedevere-bot bedevere-bot mentioned this pull request Jun 2, 2023
Open
@ericsnowcurrently ericsnowcurrently removed the request for review from a team June 2, 2023 22:28
@ericsnowcurrently ericsnowcurrently enabled auto-merge (squash) June 2, 2023 22:30
@ericsnowcurrently ericsnowcurrently merged commit e6373c0 into python:main Jun 2, 2023
21 checks passed
@ericsnowcurrently ericsnowcurrently deleted the _xxsubinterpreters-public-api-only branch June 2, 2023 23:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
skip news
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ericsnowcurrently @bedevere-bot