Skip to content

bump golangci-lint to v2.14.0 for Go 1.27.2 export data - #168

Merged
Tomer Rosenthal (torosent) merged 1 commit into
mainfrom
torosent-fix-golangci-lint-go1-27-2
Oct 12, 2026
Merged

Tomer Rosenthal (torosent) merged 1 commit into
mainfrom
torosent-fix-golangci-lint-go1-27-2

Conversation

@torosent

Copy link
Copy Markdown
Member

main has been red since Go 1.27.2 shipped. The 1.27.x leg's golangci-lint step fails typecheck with export data version 5 is greater than maximum supported version 4 (failing run). Go 1.27.2 backported unified V5 export data. golangci-lint v2.13.2 bundles golang.org/x/tools v0.49.0, which decodes only up to V4. v2.14.0 bundles v0.50.0, which decodes V5. #166 was just the first run after 1.27.x moved from 1.27.1 to 1.27.2.

Dependabot bumps the action SHA but not the version: input, so the pin now has a comment explaining when it needs a manual bump.

Validation: reproduced the CI error locally with v2.13.2 on Go 1.27.2. v2.14.0 reports 0 issues on both matrix toolchains (Go 1.27.2 and 1.25.14).

Co-authored-by: Copilot App 223556219+Copilot@users.noreply.github.com

Go 1.27.2 backported unified V5 export data. golangci-lint v2.13.2
bundles golang.org/x/tools v0.49.0, which only decodes up to V4, so the
1.27.x matrix leg fails typecheck with "export data version 5 is greater
than maximum supported version 4". v2.14.0 bundles x/tools v0.50.0,
which supports V5.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@torosent
Tomer Rosenthal (torosent) marked this pull request as ready for review October 12, 2026 21:26
Copilot AI balanced review requested due to automatic review settings October 12, 2026 21:26

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The targeted version bump addresses the verified CI failure without introducing unrelated changes.

0 open findings

What changed in this PR

Updates CI lint compatibility for Go 1.27.2 export data.

Changes:

  • Bumps golangci-lint from v2.13.2 to v2.14.0.
  • Documents why manual pin updates are required.
File Description
.github/​workflows/​pr-validation.yml Updates and documents the golangci-lint pin.

🧠 Review effort: Balanced


💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@torosent
Tomer Rosenthal (torosent) merged commit c713409 into main Oct 12, 2026
8 checks passed
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.

3 participants