fix(ci): use go-test-coverage binary directly to fix Gitea Actions #303

Merged
argoyle merged 1 commits from fix/coverage-tool-binary into main 2026-04-29 06:06:23 +00:00
Owner

Summary

  • vladopajic/go-test-coverage@v2 (v2.18.5+, released 2026-04-26/27) restructured its composite action to pass inputs via env-var mapping. Gitea act_runner doesn't expand ${{ }} expressions inside docker-action env: blocks reliably, so the literal string ${{ inputs.config }} reached the binary and broke the 'Check coverage' step.
  • Replace the action with a direct go install + binary invocation (matching the established Frostmoln pattern).
  • Use --github-action-output to expose total-coverage as a step output, replacing the manual go tool cover -func | grep | awk calculations.
  • Baseline artifact now stores the percentage directly instead of the full coverage profile.
  • Bump go directive in go.mod from 1.22.12 → 1.26.2 (matching toolchain) — we are the sole consumers of this module.

Test plan

  • prek run --all-files passes
  • CI passes on this PR
  • After merge, baseline artifact format propagates on next push to main
## Summary - `vladopajic/go-test-coverage@v2` (v2.18.5+, released 2026-04-26/27) restructured its composite action to pass inputs via env-var mapping. Gitea `act_runner` doesn't expand `${{ }}` expressions inside docker-action `env:` blocks reliably, so the literal string `${{ inputs.config }}` reached the binary and broke the 'Check coverage' step. - Replace the action with a direct `go install` + binary invocation (matching the established Frostmoln pattern). - Use `--github-action-output` to expose `total-coverage` as a step output, replacing the manual `go tool cover -func | grep | awk` calculations. - Baseline artifact now stores the percentage directly instead of the full coverage profile. - Bump `go` directive in `go.mod` from 1.22.12 → 1.26.2 (matching toolchain) — we are the sole consumers of this module. ## Test plan - [x] `prek run --all-files` passes - [ ] CI passes on this PR - [ ] After merge, baseline artifact format propagates on next push to main
argoyle added 1 commit 2026-04-29 05:39:44 +00:00
fix(ci): use go-test-coverage binary directly to fix Gitea Actions
authz_client / vulnerabilities (pull_request) Successful in 1m42s
authz_client / test (pull_request) Successful in 2m34s
pre-commit / pre-commit (pull_request) Successful in 7m47s
fd7f7fca18
Switch from broken composite action wrapper to direct go install +
binary invocation matching the established Frostmoln pattern. Use
--github-action-output to expose total-coverage as a step output.

Coverage Report

Total coverage: 97%

## Coverage Report Total coverage: **97%**
argoyle scheduled this pull request to auto merge when all checks succeed 2026-04-29 06:06:21 +00:00
argoyle merged commit f9a89b64be into main 2026-04-29 06:06:23 +00:00
argoyle deleted branch fix/coverage-tool-binary 2026-04-29 06:06:24 +00:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shiny/authz_client#303