ci: set coverage threshold to 100%, govulncheck gated behind go-build
CI / Nginx config check (push) Failing after 5s
CI / Secrets scan (push) Failing after 6s
CI / Docker compose check (push) Failing after 6s
CI / Env docs check (push) Failing after 7s
CI / Frontend deps check (push) Failing after 21s
CI / Knip (push) Has been skipped
CI / Frontend a11y check (push) Has been skipped
CI / Go build (push) Successful in 26s
CI / go mod tidy (push) Successful in 17s
CI / Frontend build (push) Successful in 1m5s
CI / Svelte strict check (push) Has been skipped
CI / Frontend QC (audit) (push) Has been skipped
CI / Frontend QC (typecheck) (push) Has been skipped
CI / Frontend QC (lint) (push) Has been skipped
CI / Go vulnerabilities (push) Successful in 1m17s
CI / Go vet (push) Successful in 1m42s
CI / Staticcheck (push) Failing after 1m58s
CI / golangci-lint (push) Successful in 2m15s
CI / Security scan (gosec) (push) Failing after 2m48s
CI / Tests (prod) (push) Has been skipped
CI / Tests (dev) (push) Has been skipped
CI / Race (prod) (push) Has been skipped
CI / Race (dev) (push) Has been skipped
CI / Nginx config check (push) Failing after 5s
CI / Secrets scan (push) Failing after 6s
CI / Docker compose check (push) Failing after 6s
CI / Env docs check (push) Failing after 7s
CI / Frontend deps check (push) Failing after 21s
CI / Knip (push) Has been skipped
CI / Frontend a11y check (push) Has been skipped
CI / Go build (push) Successful in 26s
CI / go mod tidy (push) Successful in 17s
CI / Frontend build (push) Successful in 1m5s
CI / Svelte strict check (push) Has been skipped
CI / Frontend QC (audit) (push) Has been skipped
CI / Frontend QC (typecheck) (push) Has been skipped
CI / Frontend QC (lint) (push) Has been skipped
CI / Go vulnerabilities (push) Successful in 1m17s
CI / Go vet (push) Successful in 1m42s
CI / Staticcheck (push) Failing after 1m58s
CI / golangci-lint (push) Successful in 2m15s
CI / Security scan (gosec) (push) Failing after 2m48s
CI / Tests (prod) (push) Has been skipped
CI / Tests (dev) (push) Has been skipped
CI / Race (prod) (push) Has been skipped
CI / Race (dev) (push) Has been skipped
Coverage check now requires 100% (fail on any uncovered line). govulncheck (vulns) now has needs: [go-build], placing it in the gate block alongside vet/lint/staticcheck/gosec. Test and race already depended on vulns, so the chain is: go-build → gate jobs (incl vulns) → test+race. Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-openagent) Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
This commit is contained in:
@@ -347,11 +347,11 @@ jobs:
|
||||
if [ -f coverage.out ]; then
|
||||
COVERAGE=$(go tool cover -func=coverage.out | grep total | awk '{print $3}' | sed 's/%//')
|
||||
echo "Total coverage: $COVERAGE%"
|
||||
if [ "$(echo "$COVERAGE < 50" | bc)" -eq 1 ]; then
|
||||
echo "FAIL: Coverage $COVERAGE% is below 50% threshold"
|
||||
if [ "$(echo "$COVERAGE < 100" | bc)" -eq 1 ]; then
|
||||
echo "FAIL: Coverage $COVERAGE% is below 100% threshold"
|
||||
exit 1
|
||||
fi
|
||||
echo "PASS: Coverage $COVERAGE% meets 50% threshold"
|
||||
echo "PASS: Coverage $COVERAGE% meets 100% threshold"
|
||||
else
|
||||
echo "No coverage file generated (prod build)"
|
||||
fi
|
||||
@@ -429,6 +429,7 @@ jobs:
|
||||
|
||||
vulns:
|
||||
name: Go vulnerabilities
|
||||
needs: [go-build]
|
||||
runs-on: ubuntu-latest
|
||||
defaults:
|
||||
run:
|
||||
|
||||
Reference in New Issue
Block a user