Skip to content

chore: bump golangci-lint to v2.12.2#545

Merged
Benehiko merged 1 commit into
mainfrom
chore/bump-golangci-lint
Jun 2, 2026
Merged

chore: bump golangci-lint to v2.12.2#545
Benehiko merged 1 commit into
mainfrom
chore/bump-golangci-lint

Conversation

@Benehiko
Copy link
Copy Markdown
Member

@Benehiko Benehiko commented Jun 2, 2026

Unify the two golangci-lint pins (Makefile v2.4.0 and Dockerfile ARG default v2.2.1) on v2.12.2. The newer gofumpt extra-rules flagged two naked returns in the vendored go-keychain corefoundation.go; apply the formatter's named-return fix so the lint stage stays green.

Unify the two golangci-lint pins (Makefile v2.4.0 and Dockerfile ARG default
v2.2.1) on v2.12.2. The newer gofumpt extra-rules flagged two naked returns in
the vendored go-keychain corefoundation.go; apply the formatter's named-return
fix so the lint stage stays green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown

@docker-agent docker-agent Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Assessment: 🟢 APPROVE

This is a clean, low-risk tooling bump:

  • Dockerfile and Makefile: version pins unified to v2.12.2 (previously v2.2.1 and v2.4.0 respectively)
  • corefoundation.go: two naked return statements replaced with explicit return m / return a — semantically identical, satisfying the stricter gofumpt rules in the new linter version

No logic errors, security issues, resource leaks, or behavioral changes were introduced.

@Benehiko Benehiko merged commit 4f3871e into main Jun 2, 2026
22 checks passed
@Benehiko Benehiko deleted the chore/bump-golangci-lint branch June 2, 2026 12:46
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