Skip to content

[release-1.3] Bump golangcilint to v2#1177

Merged
ahrtr merged 2 commits intoetcd-io:release-1.3from
ivanvc:bump-golangci-to-v2
Apr 16, 2026
Merged

[release-1.3] Bump golangcilint to v2#1177
ahrtr merged 2 commits intoetcd-io:release-1.3from
ivanvc:bump-golangci-to-v2

Conversation

@ivanvc
Copy link
Copy Markdown
Member

@ivanvc ivanvc commented Apr 1, 2026

Manual backport of #954 / 675a3be.

Silence linters that fail with the current state of the code, as we don't want to change the stable release code.

Trying to unblock #1158 / The Go version bump (etcd-io/etcd#21458).

Manual backport of etcd-io#954 / 675a3be.

Silence linters that fail with the current state of the code, as we
don't want to change the stable release code.

Signed-off-by: Ivan Valdes <ivan@vald.es>
@ahrtr
Copy link
Copy Markdown
Member

ahrtr commented Apr 1, 2026

/ok-to-test

This has been a dead end for several attempts. Golangci-lint fails for
every file for Windows. The easier approach, given this is a stable
rlease branch is to disable linting in Windows.

Signed-off-by: Ivan Valdes <ivan@vald.es>
@ivanvc
Copy link
Copy Markdown
Member Author

ivanvc commented Apr 15, 2026

Same treatment as #1178, disabled linting on Windows.

/cc @ahrtr

@k8s-ci-robot k8s-ci-robot requested a review from ahrtr April 15, 2026 18:28
@k8s-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: ahrtr, Elbehery, ivanvc

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@ahrtr ahrtr merged commit e1dda16 into etcd-io:release-1.3 Apr 16, 2026
36 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

4 participants