Skip to content

Commit d05dd79

Browse files
michaelmcneesclaude
andcommitted
fix: add || true to govulncheck grep to prevent set -e false failure
When all vulnerabilities are excluded, grep -vE exits 1 (no matches). Under GitHub Actions' default set -e + pipefail this aborts the step even though no actionable vulnerabilities exist. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
1 parent 29a9042 commit d05dd79

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

.github/workflows/engine-ci.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -65,7 +65,7 @@ jobs:
6565
# Fixed in: N/A — no upstream fix available. Remove exclusions when a patched release ships.
6666
OUTPUT=$(govulncheck ./... 2>&1 || true)
6767
echo "$OUTPUT"
68-
ACTIONABLE=$(echo "$OUTPUT" | grep -E '^Vulnerability #[0-9]+:' | grep -vE 'GO-2026-4887|GO-2026-4883')
68+
ACTIONABLE=$(echo "$OUTPUT" | grep -E '^Vulnerability #[0-9]+:' | grep -vE 'GO-2026-4887|GO-2026-4883' || true)
6969
if [ -n "$ACTIONABLE" ]; then
7070
echo "FAIL: unfixed actionable vulnerabilities found"
7171
exit 1

0 commit comments

Comments
 (0)