-
-
Notifications
You must be signed in to change notification settings - Fork 171
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
add codecov, tweak some spelling #384
Conversation
I think to get the report to show up in the issue, one button must be pushed, once, in the codecov app. |
Codecov Report
@@ Coverage Diff @@
## master #384 +/- ##
=========================================
Coverage ? 99.72%
=========================================
Files ? 65
Lines ? 8828
Branches ? 0
=========================================
Hits ? 8804
Misses ? 24
Partials ? 0 Continue to review full report at Codecov.
|
Adding the codecov badge does require a (non-secret) token... the markdown/rst can be copied from the settings page on codecov. |
|
When I click on "continue to full report" [1], it says the page is missing. |
Yeah, once some reports hit |
oh, merged! welp, added some badges on my push, feel free to steal! |
merged. thanks. |
A follow-up to #379 (review), this uploads coverage to
codecov
, which is slightly simpler to set up, and doesn't require a secret token.Also fixes a few spelling issues, just so we can see all the ✔️ .