Skip to content

chore(dep): update go to v1.27, golangci-lint to v2.13, and fix all the lint - #5959

Merged
Ly-Joey merged 18 commits into
google:masterfrom
Ly-Joey:chore-update-golangci-lint
Sep 3, 2026
Merged

chore(dep): update go to v1.27, golangci-lint to v2.13, and fix all the lint#5959
Ly-Joey merged 18 commits into
google:masterfrom
Ly-Joey:chore-update-golangci-lint

Conversation

@Ly-Joey

@Ly-Joey Ly-Joey commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Update Go module directives and Docker base images to Go 1.27.0.

Go 1.27 introduced a new compiler export data format (v4) for compiled standard library packages. Older versions of golangci-lint fail during typechecking when encountering Go 1.27 standard library export data, so we are updating it to v2.13.0 as well.

Also added a golangci config file to disable exhaustruct_v5 and fixed ALL the new lint errors found.

@Ly-Joey
Ly-Joey requested review from a team and michaelkedar and removed request for a team and michaelkedar September 2, 2026 01:42
Ly-Joey added a commit that referenced this pull request Sep 2, 2026
Ly-Joey added a commit that referenced this pull request Sep 2, 2026
Ly-Joey added a commit that referenced this pull request Sep 2, 2026
another-rex
another-rex previously approved these changes Sep 2, 2026
@Ly-Joey
Ly-Joey force-pushed the chore-update-golangci-lint branch 2 times, most recently from c2dde22 to 0918e24 Compare September 2, 2026 04:00
@Ly-Joey
Ly-Joey requested a review from another-rex September 2, 2026 05:16

@another-rex another-rex 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.

also update the go version in lint.yaml in .workflows

@Ly-Joey
Ly-Joey force-pushed the chore-update-golangci-lint branch from 4aa2b00 to 4853559 Compare September 2, 2026 07:14
@Ly-Joey Ly-Joey changed the title chore(dep): update go to v1.27 and golangci-lint chore(dep): update go to v1.27, golangci-lint to v2.13, and fix all the lint Sep 3, 2026
@Ly-Joey
Ly-Joey requested review from a team and michaelkedar September 3, 2026 23:26
@Ly-Joey
Ly-Joey merged commit b9b8d24 into google:master Sep 3, 2026
27 of 28 checks passed
Ly-Joey added a commit that referenced this pull request Sep 4, 2026
nolint:gosec and nolintlint are fighting so we're switching to the gosec
native ignore pattern.
Seems like our lint workflow only intermittently report the nolintlint
error, which is why #5959 was fine.
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