-
Notifications
You must be signed in to change notification settings - Fork 26
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
[DOC] Add Codecov coverage testing #127
Conversation
Anything more I can help out with for this? |
@jGaboardi I'll be sprinting on this again on Monday, would you be interested interested in another work along session like last time? :) |
I'll probably be able to do that and will absolutely join if I can. |
@jGaboardi do you recall what needs to be done here (if something)? |
Although not required, I would like to see some changes to
|
@jGaboardi can you push those? It seems you have a clear idea what to do. We'll get it out of the way and make that patch release with #140, which is well overdue. |
Should we also put in another PR for trimming down the CI matrix? |
@martinfleis, Do you want to resolve the |
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.
Thanks both!
Codecov Report
@@ Coverage Diff @@
## main #127 +/- ##
=======================================
+ Coverage 75.8% 76.4% +0.6%
=======================================
Files 19 19
Lines 1570 1570
=======================================
+ Hits 1190 1199 +9
+ Misses 380 371 -9
|
README.md
.codecov.yml
after Numpy example