nipreps / eddymotion

Open-source eddy-current and head-motion correction for dMRI.
https://nipreps.org/eddymotion
Apache License 2.0
13 stars 16 forks source link

MAINT: Revision of GHA migration #230

Closed oesteban closed 1 day ago

oesteban commented 2 days ago

Revises the GHA migration to make sure that tests run:

IMHO, the addition of tox deviates from other projects (and as far as I understand, we were not planning on using tox) and has introduced some complexity. In particular, I had to set one test to skip because of (I suspect) some minimal environment change tox introduces.

For now, I wanted to focus on the download of data and execution of all tests. Future PRs can address the removal of tox if we finally decide against it.

codecov[bot] commented 2 days ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 53.35%. Comparing base (a7880a3) to head (7fb31c8). Report is 5 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #230 +/- ## =========================================== + Coverage 32.97% 53.35% +20.37% =========================================== Files 17 17 Lines 849 849 Branches 149 143 -6 =========================================== + Hits 280 453 +173 + Misses 558 369 -189 - Partials 11 27 +16 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.