Skip to content

ci: start building cp313t wheels #1333

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
zsol merged 2 commits into from
May 25, 2025
Merged

ci: start building cp313t wheels #1333

zsol merged 2 commits into from
May 25, 2025

Conversation

@zsol
Copy link
Contributor

@zsol zsol commented May 22, 2025 •
edited
Loading

ci: start building cp313t wheels

Closes #1242.

Stack created with Sapling. Best reviewed with ReviewStack.

@zsol zsol mentioned this pull request May 22, 2025
Merged
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 22, 2025
@zsol zsol force-pushed the pr1333 branch 2 times, most recently from 3b3afc8 to 34f275f Compare May 22, 2025 19:44
@zsol zsol marked this pull request as ready for review May 22, 2025 19:44
amyreese
amyreese approved these changes May 22, 2025
View reviewed changes
Copy link
Contributor

@amyreese amyreese left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like this needs a rebase onto main since the multiprocessing PR landed.

zsol added 2 commits May 25, 2025 11:32
@zsol
Enable support for free-threading
366f3fd 
This PR:
1. marks the `libcst.native` module as free-threading-compatible
2. replaces the use of ProcessPoolExecutor with ThreadPoolExecutor if free-threaded CPython is detected at runtime

This depends on #1294 and #1289.
@zsol
ci: start building cp313t wheels
9d69496 
Closes #1242.
@zsol zsol merged commit c224665 into main May 25, 2025
61 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@amyreese amyreese amyreese approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Publish cp313t (free-threaded) wheels

4 participants

@zsol @amyreese @facebook-github-bot