ci: add Codecov coverage for Go SDK (#2773)#2793
ci: add Codecov coverage for Go SDK (#2773)#2793atharvalade wants to merge 1 commit intoapache:masterfrom
Conversation
|
can someone look into this Rust integration test ( |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #2793 +/- ##
============================================
- Coverage 68.47% 64.74% -3.73%
Complexity 656 656
============================================
Files 852 900 +48
Lines 68221 72576 +4355
Branches 59682 59692 +10
============================================
+ Hits 46714 46989 +275
- Misses 19177 23240 +4063
- Partials 2330 2347 +17
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
|
The |
ea1b5c5 to
991f60b
Compare
added |
991f60b to
db232ed
Compare
Which issue does this PR close?
Closes #2773
Rationale
Rust and Java already upload coverage to Codecov — Go SDK was generating coverage but never uploading it.
What changed?
Coverage was already produced (
reports/go-coverage.out) by the pre-merge action but never sent to Codecov. Added acodecov/codecov-action@v5upload step in_test.ymlforsdk-go, ago-coveragebaseline job inpost-merge.ymlfollowing the Java pattern (start server, run tests with coverage, upload), and registered thegoflag incodecov.ymlwhile removingforeign/go/**from the ignore list.Local Execution
AI Usage