-
Notifications
You must be signed in to change notification settings - Fork 32
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
[TST] Add dummy dataset for pytest/local testing #262
Conversation
Hello @jdkent, Thank you for updating! Cheers! There are no style issues detected in this Pull Request. 🍻 To test for issues locally, Comment last updated at 2020-11-19 21:52:51 UTC |
@@ -33,7 +33,10 @@ install_requires = | |||
|
|||
[options.extras_require] | |||
duecredit = duecredit | |||
test = coverage<5 | |||
test = | |||
coverage<5 |
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.
Can we update coverage at this point? Would you mind making a new branch with this where we use latset?
I think I need help to determine how to properly |
I would just upload coverage separately in each job. Codecov is fine receiving multiple reports. |
Codecov Report
@@ Coverage Diff @@
## master #262 +/- ##
==========================================
+ Coverage 75.16% 81.35% +6.19%
==========================================
Files 21 23 +2
Lines 1506 1459 -47
Branches 263 227 -36
==========================================
+ Hits 1132 1187 +55
+ Misses 261 244 -17
+ Partials 113 28 -85
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Sorry, I should have noted that this was bound to conflict with the big AFNI PR. Care to merge/rebase master? |
9b1a975
to
6ac620b
Compare
This pull request includes: