Skip to content

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?

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