Skip to content

feat(ci): add GitHub native code coverage upload#2

Merged
senrecep merged 2 commits into
mainfrom
feat/code-coverage
May 27, 2026
Merged

feat(ci): add GitHub native code coverage upload#2
senrecep merged 2 commits into
mainfrom
feat/code-coverage

Conversation

@senrecep
Copy link
Copy Markdown
Owner

Summary

  • vitest.config.tscobertura reporter eklendi, CI'da Cobertura XML üretilir
  • ci.ymlcode-quality: write permission eklendi
  • Node 20.x'te test:coverage çalıştırılır ve actions/upload-code-coverage@v1 ile GitHub native code coverage API'sine yüklenir
  • 18.x ve 22.x sadece npm test çalıştırır (duplicate upload önlenir)

Test plan

  • CI'da 3 Node versiyonu için build & test geçiyor
  • Node 20.x'te coverage/cobertura-coverage.xml üretiliyor
  • Coverage raporu GitHub PR'da görünüyor

🤖 Generated with Claude Code

senrecep and others added 2 commits May 27, 2026 08:23
- Add cobertura reporter to vitest coverage config
- Add code-quality:write permission to CI workflow
- Upload coverage via actions/upload-code-coverage@v1 on Node 20.x only
- Other Node versions (18.x, 22.x) run tests without coverage to avoid duplicate uploads

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Replace actions/upload-code-coverage@v1 (requires GitHub Team plan)
with codecov/codecov-action@v5. Remove code-quality:write permission.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 27, 2026

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 ☂️

@senrecep senrecep merged commit de4faa9 into main May 27, 2026
8 checks passed
@senrecep senrecep deleted the feat/code-coverage branch May 27, 2026 05:34
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.

1 participant