Skip to content

Fix lint CI workflow#25

Merged
grahamgilbert merged 1 commit into
mainfrom
fix-lint-ci
Jul 1, 2026
Merged

Fix lint CI workflow#25
grahamgilbert merged 1 commit into
mainfrom
fix-lint-ci

Conversation

@grahamgilbert

@grahamgilbert grahamgilbert commented Jul 1, 2026

Copy link
Copy Markdown
Member

Fixes the failing lint job seen on PR #24. The old workflow ran setup-go before checkout and did not pin the Go version, so Actions resolved Go 1.26 while golangci-lint v1.64.8 was built with Go 1.24, causing typecheck failures before lint could run cleanly. This updates the lint workflow to checkout first, use setup-go v6 with go.mod, and pins golangci-lint-action to the v6 line with golangci-lint v1.64.8 so the existing lint behavior is preserved. Verified locally with: GOCACHE=/tmp/default-browser-go-build GOMODCACHE=/tmp/default-browser-go-mod go test ./...; GOCACHE=/tmp/default-browser-go-build GOMODCACHE=/tmp/default-browser-go-mod go run github.com/golangci/golangci-lint/cmd/golangci-lint@v1.64.8 run.

@grahamgilbert grahamgilbert force-pushed the fix-lint-ci branch 2 times, most recently from d7c177a to 0addbc9 Compare July 1, 2026 05:20
@grahamgilbert grahamgilbert merged commit 249a321 into main Jul 1, 2026
4 checks passed
@grahamgilbert grahamgilbert deleted the fix-lint-ci branch July 1, 2026 06:01
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.

1 participant