arcus-azure / arcus.security

Security for Azure development in a breeze.
https://security.arcus-azure.net/
MIT License
39 stars 12 forks source link

chore: use codecov reporting #398

Closed stijnmoreels closed 1 year ago

stijnmoreels commented 1 year ago

Use Codecov reporting.

netlify[bot] commented 1 year ago

Deploy Preview for arcus-security canceled.

Name Link
Latest commit 7c615f9cef24abb0f3792623cd0ae1b6ee7a17f1
Latest deploy log https://app.netlify.com/sites/arcus-security/deploys/63c8f9066706740009e59053
codecov[bot] commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@733e0e9). Click here to learn what that means. Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #398 +/- ## ======================================= Coverage ? 89.85% ======================================= Files ? 20 Lines ? 1065 Branches ? 82 ======================================= Hits ? 957 Misses ? 79 Partials ? 29 ``` Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=arcus-azure). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=arcus-azure)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

stijnmoreels commented 1 year ago

What do you think of this kind of reporting? Using Codedev, we can also create a status badge afterwards with a percentage in our README.md file. Right now, because main has never run with code coverage, it only shows the percentage of the PR.

fgheysels commented 1 year ago

Code coverage will be really helpful to see what code has not been tested, and if we can live with that. But I can already see we're at a very high coverage percentage :) Great job.