Skip to content

Fix static analysis CI by using actions/setup-go#785

Merged
christophetd merged 2 commits intomainfrom
fix/static-analysis-egress-policy
Mar 25, 2026
Merged

Fix static analysis CI by using actions/setup-go#785
christophetd merged 2 commits intomainfrom
fix/static-analysis-egress-policy

Conversation

@christophetd
Copy link
Copy Markdown
Contributor

@christophetd christophetd commented Mar 25, 2026

Summary

Test plan

  • Verify the "Run Go static analysis" check passes on this PR

The setup-go-faster action (used by staticcheck-action) downloads Go
binaries from dl.google.com, which was blocked by the Harden Runner
egress policy, causing exit code 22 on all PRs.
@christophetd christophetd requested review from a team as code owners March 25, 2026 13:25
The embedded setup-go-faster@v1.14.0 in staticcheck-action downloads Go
from storage.googleapis.com/golang/ which now returns HTTP 403, causing
curl exit code 22. Use actions/setup-go@v5 separately and set
install-go: false on the staticcheck action.
@christophetd christophetd changed the title Fix static analysis CI egress policy Fix static analysis CI by using actions/setup-go Mar 25, 2026
@christophetd christophetd enabled auto-merge (squash) March 25, 2026 13:53
@christophetd christophetd merged commit c5a0a89 into main Mar 25, 2026
5 checks passed
@christophetd christophetd deleted the fix/static-analysis-egress-policy branch March 25, 2026 14:06
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