Skip to content

ci: upload coverage to Codecov#11

Merged
peczenyj merged 1 commit into
develfrom
feature/codecov
May 25, 2026
Merged

ci: upload coverage to Codecov#11
peczenyj merged 1 commit into
develfrom
feature/codecov

Conversation

@peczenyj

Copy link
Copy Markdown
Owner

Now that we have unit tests, wire Codecov into CI (mirrors the go-claimcheck setup).

What

  • CI Test step now runs task test:coverage (emits coverage.txt); the stable-Go matrix job uploads it via codecov/codecov-action (SHA-pinned v6.0.1, fail_ci_if_error: false).
  • .codecov.yml — 70% project/patch target (1% threshold), ignoring *_test.go, generated *_enumer.go, mocks/, testutil/, and the thin cmd/structalign/main.go entrypoint.
  • README codecov badge; coverage.txt/coverage.html added to .gitignore.

⚠️ Action required

Add a CODECOV_TOKEN repository secret (Settings → Secrets and variables → Actions). Until then the upload step is a no-op/soft-fail (fail_ci_if_error: false, so CI stays green). The repo also needs to be enabled on codecov.io.

Independent of the v0.3.0 refactor series; safe to merge anytime.

🤖 Generated with Claude Code

Wire Codecov into the CI workflow: the Test step now runs `task test:coverage`
(producing coverage.txt) and the stable-Go matrix job uploads it via
codecov/codecov-action (SHA-pinned v6.0.1). Adds .codecov.yml (70% project/patch
target, ignoring tests, generated *_enumer.go, mocks, testutil, and the thin
main.go entrypoint), a coverage badge, and coverage.txt to .gitignore.

Requires a CODECOV_TOKEN repository secret.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@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.

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

@peczenyj peczenyj merged commit 5a93369 into devel May 25, 2026
6 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