aleph-im / aleph-vm

Aleph.im VM execution engine
MIT License
43 stars 18 forks source link

ci/fix(test-using-pytest): ensure hatch is always installed when needed #690

Closed Psycojoker closed 2 months ago

Psycojoker commented 2 months ago

Sometime in test-using-pytest CI workflow the previous job failed, for example here, and this line with if: always() won't be able to run because hatch is installed in the previous step but this step might get skipped.

Self proofreading checklist

Changes

Always install hatch in this always step of the job.

How to test

Make a PR that will fail this job, for example by not respecting black or use act to test that locally.

Psycojoker commented 2 months ago

@hoh this is related to your #689 PR

codecov[bot] commented 2 months ago

Codecov Report

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

Project coverage is 62.69%. Comparing base (02affa3) to head (3ac7244).

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #690 +/- ## ======================================= Coverage 62.69% 62.69% ======================================= Files 69 69 Lines 6069 6069 Branches 642 642 ======================================= Hits 3805 3805 Misses 2113 2113 Partials 151 151 ```

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