From 419a4f8136919ec1116e1bd1e581d71de2b86998 Mon Sep 17 00:00:00 2001 From: Calin Martinconi Date: Wed, 3 Jun 2026 20:04:39 +0300 Subject: [PATCH] ci: disable golangci-lint cache to avoid stale-cache lint failures A restored golangci-lint-action cache produced phantom staticcheck findings (SA4023, SA5011) in files unrelated to the PR diff. Reproduced and diagnosed on #5431: local runs with the exact CI versions reported 0 issues, the job log showed a golangci cache hit, and re-running after deleting the cache passed on the same commit. Disable the cache so the linter always analyzes fresh. Refs: #5431 --- .github/workflows/go.yml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.github/workflows/go.yml b/.github/workflows/go.yml index 51afeff511a..0d8fb3b2cce 100644 --- a/.github/workflows/go.yml +++ b/.github/workflows/go.yml @@ -71,7 +71,10 @@ jobs: - name: GolangCI-Lint uses: golangci/golangci-lint-action@1e7e51e771db61008b38414a730f564565cf7c20 # v9.2.0 with: - skip-cache: false + # Disable the golangci-lint cache: a restored stale cache has produced + # phantom staticcheck findings in files unrelated to the PR diff. The + # full lint run is fast enough that correctness is worth the trade-off. + skip-cache: true version: v2.11.3 - name: Whitespace check run: make check-whitespace