Skip to content

ci: use latest golangci-lint - #77

Merged
kolyshkin merged 1 commit into
opencontainers:mainfrom
kolyshkin:ci-golangci-lint
Oct 2, 2026
Merged

kolyshkin merged 1 commit into
opencontainers:mainfrom
kolyshkin:ci-golangci-lint

Conversation

@kolyshkin

Copy link
Copy Markdown
Contributor

golangci-lint v2.12 does not support Go 1.27, which is what go-version: stable now resolves to, so the lint job fails on every PR with typecheck errors in the standard library (see e.g. this run):

.../src/crypto/internal/randutil/randutil.go:11:2: could not import math/rand/v2 (.../src/math/rand/v2/rand.go:213:17: method must have no type parameters) (typecheck)

The golangci-lint version has to be bumped manually (dependabot does not do that), while the Go version changes as soon as a new release is out. Instead of pinning, let golangci-lint-action use the latest v2 release, which is currently v2.14.0 (Go 1.27 support was added in v2.13.0).

golangci-lint v2.12 does not support Go 1.27, which is what
"go-version: stable" now resolves to, so the lint job fails
on every PR with typecheck errors in the standard library:

  .../src/crypto/internal/randutil/randutil.go:11:2: could not import
  math/rand/v2 (.../src/math/rand/v2/rand.go:213:17: method must have
  no type parameters) (typecheck)

The golangci-lint version has to be bumped manually (dependabot does
not do that), while the Go version changes as soon as a new release
is out. Instead of pinning, let golangci-lint-action use the latest
v2 release, which is currently v2.14.0 (Go 1.27 support was added
in v2.13.0).

Signed-off-by: Kir Kolyshkin <kolyshkin@gmail.com>
@kolyshkin
kolyshkin requested a review from a team as a code owner October 2, 2026 06:00
@kolyshkin
kolyshkin merged commit ddb76b0 into opencontainers:main Oct 2, 2026
15 checks passed
@kolyshkin

Copy link
Copy Markdown
Contributor Author

This is CI only and fixes CI for other PRs, so I'm merging it without approvals from other maintainers.

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