-
Notifications
You must be signed in to change notification settings - Fork 10
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
Try using uv
for worker
#896
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅ ✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #896 +/- ##
==========================================
- Coverage 98.04% 98.01% -0.04%
==========================================
Files 444 444
Lines 35435 35543 +108
==========================================
+ Hits 34744 34838 +94
- Misses 691 705 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #896 +/- ##
==========================================
- Coverage 98.04% 98.01% -0.04%
==========================================
Files 444 444
Lines 35435 35543 +108
==========================================
+ Hits 34744 34838 +94
- Misses 691 705 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. Additional details and impacted files@@ Coverage Diff @@
## main #896 +/- ##
==========================================
- Coverage 98.04% 98.01% -0.04%
==========================================
Files 444 444
Lines 35435 35543 +108
==========================================
+ Hits 34744 34838 +94
- Misses 691 705 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
✅ All tests successful. No failed tests found. @@ Coverage Diff @@
## main #896 +/- ##
==========================================
- Coverage 98.04% 98.01% -0.04%
==========================================
Files 444 444
Lines 35435 35543 +108
==========================================
+ Hits 34744 34838 +94
- Misses 691 705 +14
Flags with carried forward coverage won't be shown. Click here to find out more.
|
✅ All tests successful. No failed tests were found. 📣 Thoughts on this report? Let Codecov know! | Powered by Codecov |
Instead of using `uv` virtualenv to build the image, we want to build the wheels so that the docker app won't need to have dependency on build tools.
There's a few things that were fixed for the SQLAlchemy upgrade.
4798087
to
707d492
Compare
I tried taking inspiration from https://hynek.me/articles/docker-uv/, but hit some snags along the way.
I would need to dig a bit deeper learning how the docker setup fully functions.