chore: use uv #3042
chore: use uv #3042
Conversation
thomas-chauvet
requested review from
wjones127,
fvaleye,
roeap and
ion-elgreco
as code owners
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3042 +/- ##
==========================================
+ Coverage 72.64% 72.68% +0.03%
==========================================
Files 129 129
Lines 41573 41573
Branches 41573 41573
==========================================
+ Hits 30202 30218 +16
+ Misses 9401 9397 -4
+ Partials 1970 1958 -12 ☔ View full report in Codecov by Sentry. |
|
Some parts of CI need to be updated to use UV as well: https://github.com/delta-io/delta-rs/blob/main/.github/workflows/python_build.yml |
|
Changes look good, but can you resolve the cache dependency glob? Once CI passes we can merge it : ) |
Should be ok, I needed to checkout the repo before in the setup action. Hope it's ok :) |
|
@thomas-chauvet it's still failing, can you look at it |
|
@ion-elgreco, sorry I missed that it did run |
Just one more failure during install, you need to explicitly create a venv or install it system-wide :) |
ion-elgreco
force-pushed
the
chore/move-to-uv
branch
from
a2e53d7 to
6fc1b5c
Compare
Signed-off-by: Thomas <[email protected]> docs: fix activate venv Signed-off-by: Thomas <[email protected]> fix: pip uninstall with uv Signed-off-by: Thomas <[email protected]> chore: update CI workflows Signed-off-by: Thomas <[email protected]> fix(ci): need to checkout to use uv.lock Signed-off-by: Thomas <[email protected]> fix(ci): fix uv.lock path Signed-off-by: Thomas <[email protected]> Update .github/workflows/python_build.yml Update .github/workflows/python_build.yml
ion-elgreco
force-pushed
the
chore/move-to-uv
branch
from
6fc1b5c to
b21613b
Compare
ion-elgreco
force-pushed
the
chore/move-to-uv
branch
from
41630e0 to
3f9e7e5
Compare
ion-elgreco
force-pushed
the
chore/move-to-uv
branch
from
3f9e7e5 to
4518b2f
Compare
github-merge-queue
bot
removed this pull request from the merge queue due to failed status checks
|
Thanks for the PR, didn't have time to check on it last week. |
Description
Add
uvfor python dependency management and building tool.Related Issue(s)