-
Notifications
You must be signed in to change notification settings - Fork 302
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.50.1 #425
Conversation
with: | ||
go-version: 1.18.x | ||
go-version-file: go.mod |
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.
Looking at what the action does, I don't think this is going to have any effect except if ruleguard is configured. It should be OK.
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.
looks OK to me.
makefile.common comes from prometheus/prometheus and should be update here |
1d01170
to
1951919
Compare
I've rebased the PR, should be good to go I think? |
Signed-off-by: Luca Comellini <luca.com@gmail.com>
1951919
to
710bde7
Compare
@roidelapluie can it be merged? |
Fixed in #447 |
Updates:
golangci-lint aligning CI and Makefilev3.3.1
v3
to be able to read Go version fromgo.mod