-
Notifications
You must be signed in to change notification settings - Fork 295
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🌱 Bump golangci-lint to v1.54.1 #2230
🌱 Bump golangci-lint to v1.54.1 #2230
Conversation
Signed-off-by: killianmuldoon <[email protected]>
@@ -23,5 +23,5 @@ jobs: | |||
- name: golangci-lint | |||
uses: golangci/golangci-lint-action@639cd343e1d3b897ff35927a75193d57cfcba299 # tag=v3.6.0 | |||
with: | |||
version: v1.53.3 | |||
version: v1.54.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@chrischdi do you want to include this is in the grouped CI cherry-picks? #2228
We might be able to auto cherry-pick it easily after that one merges across branches though
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's keep it separate 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would avoid cherry-picking back golangci-lint bumps if not strictly necessary
(Just in general, because if we don't want to actively fix linter findings on release branches it doesn't make a lot of sense, except merge conflicts for other cherry-picks)
/lgtm |
Thx! |
LGTM label has been added. Git tree hash: e46eb027d7645b9028d7672b64fe9d36e9623718
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: sbueringer The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Bump golangci-lint to v1.54.1.
The main feature here is go v1.21 support. Notes:
v1.54.0: golangci/[email protected] (release)
v1.54.1: golangci/[email protected] (release)