-
Notifications
You must be signed in to change notification settings - Fork 23
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update setup-miniconda actions configuration #420
base: master
Are you sure you want to change the base?
Update setup-miniconda actions configuration #420
Conversation
@duncanmmacleod did the cache somehow get fixed? Also, my comments from the other thread in |
@duncanmmacleod Similar to my comment over on |
a27f4f3
to
57c5e86
Compare
I think that might be a bogus statement in the docs. In the latest pipeline all of the See in particular the |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #420 +/- ##
==========================================
- Coverage 49.64% 49.27% -0.38%
==========================================
Files 60 60
Lines 8823 8789 -34
==========================================
- Hits 4380 4330 -50
- Misses 4443 4459 +16
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
|
||
- name: Configure conda | ||
uses: conda-incubator/setup-miniconda@v3 | ||
with: | ||
auto-update-conda: true | ||
channels: conda-forge | ||
activate-environment: test |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
For consistency, can we revert the change of this line? By default test
is activated.
activate-environment: test | |
auto-update-conda: true |
This PR updates the setup-miniconda action configuration to restore cache functionality.