Skip to content

add free-threaded CI#1312

Merged
zsol merged 1 commit intoInstagram:mainfrom
ngoldbaum:free-threaded-ci
Mar 12, 2025
Merged

add free-threaded CI#1312
zsol merged 1 commit intoInstagram:mainfrom
ngoldbaum:free-threaded-ci

Conversation

@ngoldbaum
Copy link
Contributor

xref #1242

passes CI on my fork: https://github.com/ngoldbaum/LibCST/actions/runs/13817416645

Is adding dev-without-jupyter in pyproject.toml ok? It seemed like the cleanest way to handle avoiding jupyter. You could also split dev into tests and docs but I'm not sure how to sort the dependencies.

The windows 3.13t CI is pretty slow but I think most of that is just installing Python and once the cache populates it should be faster.

I'm pinning to an unreleased version of setup-python - hopefully Github does a release with free-threaded support soon.

@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 Mar 12, 2025
Copy link
Contributor

@zsol zsol left a comment

Choose a reason for hiding this comment

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

nice!

@zsol zsol merged commit d580469 into Instagram:main Mar 12, 2025
31 checks passed
# Install build-system.requires dependencies
pip install setuptools setuptools-scm setuptools-rust wheel
# Jupyter is annoying to install on free-threaded Python
pip install -e .[dev-without-jupyter] --no-build-isolation
Copy link
Contributor

Choose a reason for hiding this comment

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

oh i just noticed this, would CI still work without --no-build-isolation?

ngoldbaum added a commit to ngoldbaum/LibCST that referenced this pull request Apr 8, 2025
jorenham pushed a commit to jorenham/LibCST that referenced this pull request Feb 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

3 participants