Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Add coverage badge #51

Closed
edwardleardi opened this issue Mar 10, 2021 · 2 comments · Fixed by #68
Closed

Add coverage badge #51

edwardleardi opened this issue Mar 10, 2021 · 2 comments · Fixed by #68
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@edwardleardi
Copy link
Owner

Our beautiful 100% test coverage means something for our developer users.
@edwardleardi edwardleardi added documentation Improvements or additions to documentation enhancement New feature or request labels Mar 10, 2021
@edwardleardi
Copy link
Owner Author

Activated https://coveralls.io/repos for this repo but it seems coverage.py doesn't support outputting info to lcov.info file which is needed by Coveralls. Will reinvestigate this later.

@edwardleardi edwardleardi mentioned this issue Mar 11, 2021
Merged
20 tasks
@edwardleardi
Copy link
Owner Author

Found coveralls-python which makes it much easier to integrate GH Actions with Coveralls:

There is a strange bug though with Coveralls when using GH Actions: TheKevJames/coveralls-python#252. The solution seems to be to use this flag with the coveralls command: coveralls --service=github.

Access Coverall stats here: https://coveralls.io/github/edwardleardi/nourish

PR for this here: #68

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Link GH Actions to Coveralls edwardleardi/nourish
1 participant
@edwardleardi