-
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
Onboard repo code coverage metrics #93
Onboard repo code coverage metrics #93
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #93 +/- ##
============================================
+ Coverage 84.53% 86.58% +2.04%
- Complexity 228 271 +43
============================================
Files 61 64 +3
Lines 1371 1491 +120
Branches 56 58 +2
============================================
+ Hits 1159 1291 +132
+ Misses 189 177 -12
Partials 23 23 ☔ View full report in Codecov by Sentry. |
411c0b4
to
d9920c0
Compare
d9920c0
to
ccf5dbd
Compare
3f1a6f1
to
eb14664
Compare
src/main/java/org/opensearchmetrics/model/codecov/CodeCovResponse.java
Outdated
Show resolved
Hide resolved
src/main/java/org/opensearchmetrics/model/codecov/CodeCovResult.java
Outdated
Show resolved
Hide resolved
Seems like we need to increase the test coverage of this PR? |
Let me see @peterzhuamazon, the codecov complains missing coverage for single lines in middle of the code, |
0f1824d
to
c6e8e38
Compare
src/main/java/org/opensearchmetrics/metrics/release/CodeCoverage.java
Outdated
Show resolved
Hide resolved
5a9155a
to
d349f2c
Compare
Signed-off-by: Prudhvi Godithi <[email protected]>
d349f2c
to
3a1cffd
Compare
Description
Onboard repo code coverage metrics and mechanism to link with existing release metrics.
Issues Resolved
Part of #51 and #90
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.