DM-44144: Make sconsUtils standalone/pip-installable#125
DM-44144: Make sconsUtils standalone/pip-installable#125
Conversation
taranu
force-pushed
the
tickets/DM-44144
branch
5 times, most recently
from
07c8ac6 to
dbe0854
Compare
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #125 +/- ##
===========================================
+ Coverage 69.23% 79.48% +10.25%
===========================================
Files 3 3
Lines 39 39
Branches 4 4
===========================================
+ Hits 27 31 +4
+ Misses 9 4 -5
- Partials 3 4 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
These inheritance diagrams seem to be completely broken on pipelines.lsst.io anyway.
This is when running with pytest.
taranu
left a comment
taranu
left a comment
There was a problem hiding this comment.
Looks reasonable, assuming it all works.
|
|
||
| - name: Run tests | ||
| run: | | ||
| pytest --ignore=tests/test_simple.py -r a -v -n 3 |
There was a problem hiding this comment.
I see there is a comment in test_simple.py explaining that they must be run with scons, but perhaps a comment here would be useful too. And perhaps it should be called test_with_scons.py or something more obvious?
| - name: Update pip/wheel infrastructure | ||
| run: | | ||
| python -m pip install --upgrade pip uv | ||
| uv pip install --system wheel |
There was a problem hiding this comment.
I'm vaguely aware that uv is supposed to be better than pip alone but is uv pip basically just for speed like mamba vs conda used to be, or are there other advantages?
2 participants
No description provided.