Added flags to coverage upload depending if it is a PR or a push and …#3982
Added flags to coverage upload depending if it is a PR or a push and …#3982ghandic wants to merge 2 commits into
Conversation
…updated badge icon to point only to master flag
Codecov Report
@@ Coverage Diff @@
## master #3982 +/- ##
===========================================
+ Coverage 94.36% 100.00% +5.63%
===========================================
Files 478 410 -68
Lines 12324 10281 -2043
===========================================
- Hits 11629 10281 -1348
+ Misses 695 0 -695
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
|
📝 Docs preview for commit 9695df2 at: https://615a36184dde926297f018bc--fastapi.netlify.app |
|
Thanks for your interest @ghandic! I was checking that there's no need to use flags, it's possible to use the same current badges pointing to But still, it seems that leaving them as they are they already point to master. I just created a broken PR here: #4887 removing lots of tests, that would reduce the coverage. But the badges still seem to show 100%, so it seems that by default they show the percentage of Given that, I'll close this PR now, but thanks for the effort! |
…updated badge icon to point only to master flag