Skip to content

Cache golangci-lint binary in addition to versioned symlink#2438

Merged
stefanvanburen merged 1 commit intomainfrom
svanburen/cache-golangci-lint-binary
Apr 27, 2026
Merged

Cache golangci-lint binary in addition to versioned symlink#2438
stefanvanburen merged 1 commit intomainfrom
svanburen/cache-golangci-lint-binary

Conversation

@stefanvanburen
Copy link
Copy Markdown
Member

Noticed this wasn't being fully cached. The Makefile installs the golangci-lint binary to .tmp/golangci-lint via GOBIN, then creates a versioned symlink at .tmp/golangci-lint-. The previous cache path only matched the symlink glob, leaving the actual binary uncached and requiring a fresh install on every run.

Ref: https://github.com/bufbuild/plugins/actions/runs/24997932372/job/73199781628#step:13:13

Noticed this wasn't being fully cached. The Makefile installs the
golangci-lint binary to .tmp/golangci-lint via GOBIN, then creates
a versioned symlink at .tmp/golangci-lint-<version>. The previous cache
path only matched the symlink glob, leaving the actual binary uncached
and requiring a fresh install on every run.

Ref: https://github.com/bufbuild/plugins/actions/runs/24997932372/job/73199781628#step:13:13
@stefanvanburen stefanvanburen merged commit 235f99e into main Apr 27, 2026
4 checks passed
@stefanvanburen stefanvanburen deleted the svanburen/cache-golangci-lint-binary branch April 27, 2026 14:05
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