-
Notifications
You must be signed in to change notification settings - Fork 13
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
chore: use coverlet code coverage #396
chore: use coverlet code coverage #396
Conversation
✅ Deploy Preview for arcus-security canceled.
|
…/stijnmoreels/arcus.security into chore/use-coverlet-code-coverage
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.
Good job extending the test-suite based on the coverage results. (Although it could've been done in a separate PR).
Excluded the test functionality and the provider projects as this last one is mostly integration stuff towards other environments (i.e. Azure Key Vault, HashiCorp Vault...) and these are tested in integration tests. |
Could be in a diff PR, but I added 80% as a threshold to be above, and wanted to be above that here. 👍 |
Use Coverlet code coverage as GitHub action.
Relates to arcus-azure/arcus#264