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 CI step #1371

Open
wants to merge 11 commits into
base: main
Choose a base branch
from
Open

Use uv for CI step #1371

wants to merge 11 commits into from

Conversation

ekzhang
Copy link
Member

@ekzhang ekzhang commented Feb 20, 2024

Also cleaning up the CI a bit in general

@ekzhang
Copy link
Member Author

ekzhang commented Feb 20, 2024

It doesn't work due to some protobuf issue on Windows 🤔

image

@ekzhang
Copy link
Member Author

ekzhang commented Feb 20, 2024

I removed the cache in this case. Honestly this is only a little bit faster than our existing CI, since that one caches installed packages pretty well. We can merge it later when the Windows issue is fixed but I won't look at it more for the time being.

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.

None yet

1 participant