-
Notifications
You must be signed in to change notification settings - Fork 427
Bump golangci-lint to 2.6.0 #3688
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
base: main
Are you sure you want to change the base?
Conversation
65a30ca to
c05d4d7
Compare
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
c05d4d7 to
47b97da
Compare
On-behalf-of: @SAP [email protected]
On-behalf-of: @SAP [email protected]
On-behalf-of: @SAP [email protected]
47b97da to
52db8f2
Compare
|
@xrstf: The following test failed, say
DetailsInstructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
Summary
This is just maintenance. I also bumped logcheck because 0.9 doesn't work on Go 1.25 systems.
gocritic now complains about the misplaced Deprecation notices, and after fixing them, I had to adjust a bit of code in our codebase. However I did not remove the deprecated CreateSheriff, even though it was meant to be deprecated allll the way back in #1642 (2022). But since that never worked, nobody noticed and so I would declare today the official deprecation day 😁
What Type of PR Is This?
/kind chore
Release Notes