Skip to content

CI: Upload coverage report to Codecov#3

Merged
hayabusa-cloud merged 1 commit into
mainfrom
ci-codecov
Dec 14, 2025
Merged

CI: Upload coverage report to Codecov#3
hayabusa-cloud merged 1 commit into
mainfrom
ci-codecov

Conversation

@hayabusa-cloud

Copy link
Copy Markdown
Owner

Summary

This PR fixes the spin CI Codecov check by uploading the generated Go coverage profile to Codecov during the coverage job.

What changed

  • Added a Codecov upload step to the coverage-linux-amd64 job in spin/.github/workflows/ci.yml.
  • The workflow still enforces 100% coverage for public packages and uploads the coverage file as a GitHub artifact as before.

Why

CI already produces coverage-amd64.out, but without uploading it Codecov can’t compute/report coverage for the commit/PR, causing the Codecov check to fail or remain pending.

Testing

  • CI-only change. Verified logic: coverage file is generated before upload (coverage-amd64.out).

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

@hayabusa-cloud hayabusa-cloud marked this pull request as ready for review December 14, 2025 16:47
@hayabusa-cloud hayabusa-cloud merged commit c421d82 into main Dec 14, 2025
12 checks passed
@hayabusa-cloud hayabusa-cloud deleted the ci-codecov branch December 14, 2025 16:48
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