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

Use uv for installing dependencies in CI on Linux-only workflows #466

Merged
merged 2 commits into from
Mar 10, 2024

Conversation

AlexWaygood
Copy link
Collaborator

@AlexWaygood AlexWaygood commented Mar 10, 2024

It's currently taking 24s to install dependencies on Windows in CI, and only 7s to actually run the tests when the dependencies have been installed.

Also, bump a few Python versions in our workflow files, while we're at it.

This comment has been minimized.

@AlexWaygood
Copy link
Collaborator Author

AlexWaygood commented Mar 10, 2024

Hmm, not as much of a speedup as I was hoping for, really. Still pretty slow to install dependencies on Windows :/

Not really sure if this is worth the extra complexity for us right now, but interested to hear other people's opinions.

@srittau
Copy link
Collaborator

srittau commented Mar 10, 2024

It it's not a considerable speedup, I'd keep pip for now. Maybe at some point there will be an easier way to install uv (maybe using a GitHub Action?).

@AlexWaygood AlexWaygood deleted the uv branch March 10, 2024 14:08
@AlexWaygood AlexWaygood restored the uv branch March 10, 2024 16:59
@AlexWaygood AlexWaygood reopened this Mar 10, 2024
@AlexWaygood
Copy link
Collaborator Author

There's still pretty impressive speedups (and not nearly the same added complexity) for the jobs that don't run on Windows. I'll merge just those change for now.

@AlexWaygood AlexWaygood changed the title Use uv for installing dependencies in CI Use uv for installing dependencies in CI on Linux-only workflows Mar 10, 2024
Copy link

This change has no effect on typeshed. 🤖🎉

@AlexWaygood AlexWaygood merged commit 1f311d2 into main Mar 10, 2024
21 of 40 checks passed
@AlexWaygood AlexWaygood deleted the uv branch March 10, 2024 17:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants