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 to install dependencies in CI #419
base: main
Are you sure you want to change the base?
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #419 +/- ##
==========================================
+ Coverage 60.09% 60.12% +0.02%
==========================================
Files 23 23
Lines 1466 1467 +1
==========================================
+ Hits 881 882 +1
Misses 585 585
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
@@ -1,7 +1,7 @@ | |||
from pathlib import Path | |||
|
|||
import pytest | |||
import yaml | |||
from ruamel.yaml import YAML |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We've been depending on pyyaml
for tests by accident, and did not notice since it got installed together with pre-commit. Let's use ruamel since that's what is used in the aiidalab package.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@superstar54 could you take a look? This might be beneficial for QeApp as well.
This reverts commit 4cd0c59.
uv is the new cool kid in the town of Python packaging, from the creators of ruff. It currently serves as a much faster drop-in replacement for pip.
For the
aiidalab
package, this drops installation time from ~15s to ~2s, which nearly halves the workflow time since the tests we have here are pretty quick.Couple other small changes (feel free to push back)
remove bumpver and pre-commit from direct dev dependencies, as proposed on Discourse