-
Notifications
You must be signed in to change notification settings - Fork 4
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
Feature/coverage #19
Feature/coverage #19
Conversation
Hoping this will work better for coverage
…eck notebook output.
Making very small changes to identify why tests stopped working previously.
Maybe it broke the pytest installation previously?
Notable shortcoming of this PR is that I never found a way to get the coverage of the notebooks themselves. However, I am able to get coverage statistics for all of their dependencies and check them for consistent output, which I think is a good start. |
Codecov Report
@@ Coverage Diff @@
## develop #19 +/- ##
==========================================
Coverage ? 65.49%
==========================================
Files ? 12
Lines ? 1339
Branches ? 131
==========================================
Hits ? 877
Misses ? 434
Partials ? 28 Continue to review full report at Codecov.
|
Does your pull request resolve or partially resolve an issue?
Yes.
If Yes, which issue?
Fully resolves #16
Does your pull request implement code improvements?
No.
Does your pull request implement any breaking changes?
No.
If breaking changes are implemented, please describe:
Testing:
This pull request: