Skip to content

fix: use lcov format and token for Codecov upload#106

Merged
hydro13 merged 1 commit intomainfrom
fix/codecov-coverage-format
Apr 11, 2026
Merged

fix: use lcov format and token for Codecov upload#106
hydro13 merged 1 commit intomainfrom
fix/codecov-coverage-format

Conversation

@hydro13
Copy link
Copy Markdown
Owner

@hydro13 hydro13 commented Apr 11, 2026

Summary

  • Add lcov reporter to vitest coverage config — Codecov cannot parse json-summary, which is why the badge shows 0%
  • Update verify.yml to upload coverage/lcov.info instead of coverage-summary.json
  • Pass CODECOV_TOKEN secret to codecov-action@v4 (required for authentication)

Test plan

  • CI verify.yml passes
  • Codecov badge updates from 0% to ~44% after merge

- Add lcov reporter to vitest coverage config (Codecov can't parse json-summary)
- Point verify.yml at coverage/lcov.info instead of coverage-summary.json
- Pass CODECOV_TOKEN secret to the upload action (required by v4)
@codecov-commenter
Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

ℹ️ You can also turn on project coverage checks and project coverage reporting on Pull Request comment

Thanks for integrating Codecov - We've got you covered ☂️

@hydro13 hydro13 merged commit be75b5a into main Apr 11, 2026
4 checks passed
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