Closed aperezhortal closed 2 years ago
Merging #257 (092e0e8) into master (7213c95) will not change coverage. The diff coverage is
n/a
.
@@ Coverage Diff @@
## master #257 +/- ##
=======================================
Coverage 80.92% 80.92%
=======================================
Files 142 142
Lines 10759 10759
=======================================
Hits 8707 8707
Misses 2052 2052
Flag | Coverage Δ | |
---|---|---|
unit_tests | 80.92% <ø> (ø) |
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 7213c95...092e0e8. Read the comment docs.
@aperezhortal Nice, I was just doing the same over at h5netcdf
, but using the old style downloading via snakepit (as wradlib
does). And then I see this new provisioning thing popping up in your commits. That saved me quite some keyboard time. Just wanted to say thanks :grinning:
@aperezhortal you wanna merge this or is it still work in progress?
This PR takes #256 one step further using the
mamba-org/provision-with-micromamba@main
action to create the environment, instead ofconda-incubator/setup-miniconda@v2
Although the computational time saved is minimal (a minute at the most), it simplifies a little bit the workflow specification, condensing the previous "miniconda installation" and "environment creation" task into a single one.
More information on micromamba: https://github.com/mamba-org/mamba#micromamba