sfmig commented 2 months ago

Description

What is this PR

[ ] Bug fix
[ ] Addition of a new feature
[x] Other

Why is this PR needed? We are trying to speed up tests in CI. This PR adds a cache for the cellfinder workflow tests that run in CI.

What does this PR do? Adds a cache step to the test job that runs in CI.

References

65

How has this PR been tested?

Checks for this PR should pass.

Is this a breaking change?

No.

Does this PR require an update to the documentation?

No.

Checklist:

[ n/a] The code has been tested locally
[ n/a] Tests have been added to cover all new functionality (unit & integration)
[ n/a] The documentation has been updated to reflect any changes
[ x] The code has been formatted with pre-commit

codecov[bot] commented 2 months ago

Codecov Report

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

Project coverage is 84.45%. Comparing base (b5f62ef) to head (2a7873c). Report is 1 commits behind head on main.

Additional details and impacted files

```diff @@ Coverage Diff @@ ## main #102 +/- ## ========================================== + Coverage 79.38% 84.45% +5.06% ========================================== Files 18 17 -1 Lines 917 862 -55 ========================================== Hits 728 728 + Misses 189 134 -55 ```

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

adamltyson commented 2 months ago

@sfmig, naively, shouldn't this speed up the tests?