-
Notifications
You must be signed in to change notification settings - Fork 5
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
Our build pipelines should be extended with code-coverage reports #264
Comments
I agree. We shouldn't be too focused on this, as often integration tests cover the parts that unit tests can't cover and may not show in the results. |
I think code-coverage is very helpfull in identifying parts of code that have not been tested, and deciding if we should add a test for it, or not. The tool I'm mainly using is coverlet. But I'm open for other tools as well. |
Can we have a status badge or something like that for the code coverage as well, or would that be too much ? |
Oh, yeah, I would like that 😄 ! |
Will probably have to do a little GitHub hack to make this work: https://github.com/marketplace/actions/net-code-coverage-badge or https://github.com/marketplace/actions/opencover-badge-generator Otherwise, I'm thinking of using Codecov instead, as it has more GitHub-default support. |
I think it might be helpful that our pipelines also report code coverage.
I know we have a lot of tests already, but those code coverage reports would give us insights on which parts of the code-base are not covered by tests, and then we can see and decide if we can live with the fact that a particular set of lines of code is not tested, or if we should add additional tests.
The text was updated successfully, but these errors were encountered: