Add cache for pooch on CI for Linux jobs#8235
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8235 +/- ##
==========================================
+ Coverage 93.02% 93.08% +0.05%
==========================================
Files 701 701
Lines 63177 63177
==========================================
+ Hits 58769 58806 +37
+ Misses 4408 4371 -37 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
| uses: actions/cache@0400d5f644dc74513175e3cd8d07132dd4860809 # v4 | ||
| with: | ||
| path: ${{ env.POOCH_HOME }} | ||
| key: pooch-v |
There was a problem hiding this comment.
| key: pooch-v | |
| key: pooch-v1 |
?
But, why are we using a versioned key?
There was a problem hiding this comment.
A need to manually invalidate the cache.
But Pooch already has hashes...
|
Once tox-dev/platformdirs#375 will be merged and released, the cache will also work on macOS. But we can merge this, as it will increase test stability. |
Looking good now. Should we open an issue to track, or will the cache work automatically? |
It will happen automatically. There is a discussion about what to do for Windows. |
# References and relevant issues same as napari/napari#8235 # Description Add cache for resources required for CI. In docs we are using only Linux, so it will work for all workflows.
References and relevant issues
https://napari.zulipchat.com/#narrow/channel/298358-working-group-documentation/topic/docs.20CI
Description
To reduce the fragility of our test suite, add a cache. This will reduce the chance that the test is failing because of a network error.