Skip to content

fix: exclude vendored Go packages from coverage profile#361

Merged
tomasz-tomczyk merged 1 commit intomainfrom
fix-codecov-coverage
Apr 25, 2026
Merged

fix: exclude vendored Go packages from coverage profile#361
tomasz-tomczyk merged 1 commit intomainfrom
fix-codecov-coverage

Conversation

@tomasz-tomczyk
Copy link
Copy Markdown
Owner

Summary

  • Filters node_modules/flatted/golang/ out of go test coverage profile — it's a vendored Go package with 0% coverage that was dragging down the Codecov number vs what go test reports per-package
  • Adds generated files (gen_integration_hashes.go, integration_hashes_gen.go) to codecov ignore

Test plan

  • CI passes
  • Codecov number should match the ~58.5% that go test reports for the main package

🤖 Generated with Claude Code

go test ./... discovers a Go package inside node_modules/flatted/ which
reports 0% and drags down Codecov numbers. Filter it out via go list.
Also ignore generated files in codecov.yml.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 25, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 60.97%. Comparing base (5703e40) to head (7fabfaa).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #361      +/-   ##
==========================================
- Coverage   60.98%   60.97%   -0.02%     
==========================================
  Files          18       18              
  Lines        7807     7807              
==========================================
- Hits         4761     4760       -1     
  Misses       2619     2619              
- Partials      427      428       +1     
Flag Coverage Δ
e2e 34.89% <ø> (-0.02%) ⬇️
unit 54.78% <ø> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tomasz-tomczyk tomasz-tomczyk changed the title Exclude vendored Go packages from coverage profile fix: exclude vendored Go packages from coverage profile Apr 25, 2026
@tomasz-tomczyk tomasz-tomczyk merged commit b62fbac into main Apr 25, 2026
6 checks passed
@tomasz-tomczyk tomasz-tomczyk deleted the fix-codecov-coverage branch April 25, 2026 11:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants