-
Notifications
You must be signed in to change notification settings - Fork 492
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
CI: add simple codecov config #3997
Conversation
FWIW, this is the configuration we came up with for with Indexer: https://github.com/algorand/indexer/blob/develop/.codecov.yml |
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.
LGTM
Interested to see the output of the coverage checks. You might want to set the patch check to informational as well unless you want that failure to block reviews since I believe both patch and project are configured to run by default.
Codecov Report
@@ Coverage Diff @@
## master #3997 +/- ##
==========================================
+ Coverage 49.77% 54.46% +4.68%
==========================================
Files 409 390 -19
Lines 69157 48500 -20657
==========================================
- Hits 34426 26415 -8011
+ Misses 31011 19863 -11148
+ Partials 3720 2222 -1498
Continue to review full report at Codecov.
|
thanks @Eric-Warehime I added |
Summary
Since the codecov app was enabled, it is showing up in PR status checks. This adds a quick configuration based on https://docs.codecov.com/docs/commit-status that we can refine more later.
Test Plan
Status check should pick up changes.