-
-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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.56.1 #2842
Conversation
gaby
commented
Feb 10, 2024
- Bump golangci-lint to v1.56.1
@ReneWerner87 golangci-lint changed quite a few things in v1.56.0, I plan on fixing those. The test failing is a different issue. |
And just to say it out loud, 1.56 added https://github.com/jjti/go-spancheck which has to do with opentelemetry, so we don't need it right now. |
This was way more changes that I thought. For some reason golangci-lint stops checking after N amount of errors. So i had to constantly be running it. Turns out there was over +100 warnings |
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.
one small comment, thanks for fixing all of these