-
Notifications
You must be signed in to change notification settings - Fork 52
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
[MRG] Update GitHub actions #758
Conversation
@ntolley @jasmainak @rythorpe |
bcca482
to
84b7aa8
Compare
84b7aa8
to
c167263
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## master #758 +/- ##
==========================================
+ Coverage 92.67% 92.69% +0.02%
==========================================
Files 27 27
Lines 4928 4969 +41
==========================================
+ Hits 4567 4606 +39
- Misses 361 363 +2
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
Actually we don't need to meet for this if one of you can do it following these directions. Note the token is needed for PRs that come from within the org. It switches to token-less uploading if the PR is coming from a forked repository. See more information here. |
I'll try to help if I can find a moment tomorrow ... |
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.
@jasmainak looks like we need access to the jonescompneurolab
account to get the secret generated by codecov.
In any case this PR doesn't really depend on that update so I'm going to go ahead and merge.
Runners were no longer finding the conda base version preventing gh runners to build.
setup-miniconda
checkout