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

Switch to uv #663

Merged
merged 1 commit into from
Apr 6, 2024
Merged

Switch to uv #663

merged 1 commit into from
Apr 6, 2024

Conversation

gadomski
Copy link
Member

@gadomski gadomski commented Apr 2, 2024

Related Issue(s):

Description:

From my reading, it looks like the caching doesn't get us much, so I removed it.

PR Checklist:

  • Code is formatted
  • Tests pass
  • Changes are added to the CHANGELOG

@gadomski gadomski requested a review from jsignell April 2, 2024 17:14
@gadomski gadomski self-assigned this Apr 2, 2024
@gadomski gadomski force-pushed the issues/662-uv branch 3 times, most recently from 8b523a8 to 3449195 Compare April 2, 2024 17:35
From my reading, it looks like the caching doesn't get us much, so I removed it.
Copy link
Member

@jsignell jsignell left a comment

Choose a reason for hiding this comment

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

Yeah looks good! Took a bit less time to run than latest actions on main.

@gadomski gadomski merged commit fa4206f into main Apr 6, 2024
18 checks passed
@gadomski gadomski deleted the issues/662-uv branch April 6, 2024 13:23
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.

Use uv for the min version check
2 participants