ci: add Codecov coverage for Go SDK#2793
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 71.51% 68.05% -3.47%
Complexity 708 708
============================================
Files 970 1018 +48
Lines 78195 82550 +4355
Branches 60006 60014 +8
============================================
+ Hits 55921 56179 +258
- Misses 19941 24018 +4077
- Partials 2333 2353 +20
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
db232ed to
66aa9a3
Compare
|
looks like there's a flaky test related to Elasticsearch |
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