Skip to content

Modernize CodeQL and golangci-lint action versions#67

Merged
dolph merged 1 commit into
mainfrom
claude/fix-issue-27-modernize-ci-actions
May 18, 2026
Merged

Modernize CodeQL and golangci-lint action versions#67
dolph merged 1 commit into
mainfrom
claude/fix-issue-27-modernize-ci-actions

Conversation

@dolph
Copy link
Copy Markdown
Owner

@dolph dolph commented May 18, 2026

Summary

The CodeQL and golangci-lint workflows were pinned to action versions GitHub has retired:

  • actions/checkout@v2 (Node 12-based, retired)
  • github/codeql-action/{init,autobuild,analyze}@v1 (v1 retired by GitHub)
  • golangci/golangci-lint-action@v2 (from 2021)

These jobs were either silently failing or running in degraded mode, producing a false static-analysis signal on every PR.

Changes:

  • Bump actions/checkout to @v6 in both workflows (matches the existing ci.yml convention).
  • Bump github/codeql-action/* to @v3 (current stable major).
  • Bump golangci/golangci-lint-action to @v6.
  • Pin golangci-lint to v1.62.0. version: latest picks up new linters on every run, which can break unrelated PRs; a fixed version is deterministic and bumped intentionally.
  • Add actions/setup-go@v6 to the lint workflow, reading the Go version from go.mod so the lint job exercises the same toolchain as the test job (Go 1.20 after Bubble errors up from worker goroutines; exit non-zero on failure #66).

Test plan

  • gofmt -l . — no output
  • go vet ./... — no output
  • go build ./... — no output
  • go test -race ./... — PASS
  • CI lint job runs to completion on this PR (will be visible after push)
  • CodeQL job runs to completion on this PR

Closes #27


Generated by Claude Code

The CodeQL and golangci-lint workflows were pinned to action versions
that GitHub has retired:

- actions/checkout@v2 (Node 12-based, retired)
- github/codeql-action/{init,autobuild,analyze}@v1 (v1 retired by GitHub)
- golangci/golangci-lint-action@v2 (from 2021)

These jobs were either silently failing or running in degraded mode,
producing a false static-analysis signal on every PR.

Changes:

- Bump actions/checkout to @v6 (matches the existing ci.yml convention).
- Bump github/codeql-action/* to @V3 (current stable major).
- Bump golangci/golangci-lint-action to @v6.
- Pin golangci-lint to v1.62.0. 'version: latest' picks up new linters
  on every run, which can break unrelated PRs; a fixed version is
  deterministic and bumped intentionally.
- Add actions/setup-go@v6 to the lint workflow, reading the Go version
  from go.mod so the lint job exercises the same toolchain as the test
  job.

Closes #27
@dolph dolph added the release:skip label May 18, 2026 — with Claude
@dolph dolph merged commit 5ab94c4 into main May 18, 2026
2 checks passed
@dolph dolph deleted the claude/fix-issue-27-modernize-ci-actions branch May 18, 2026 03:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CodeQL and golangci-lint workflows use retired/EOL action versions

2 participants