-
Notifications
You must be signed in to change notification settings - Fork 183
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
Fix lint: move to Github Action #658
Conversation
Now, golangci-lint appears to succeed but then "hangs". Any ideas, @liangyuanpeng or @cheftako ? |
golangci-lint supports go1.22 since v1.56.0, per golangci/golangci-lint#4273. Switch to using Github Actions for lint check.
The lint authors recommend using Github Actions: https://golangci-lint.run/welcome/install/#ci-installation After we get this in place, we can remove the problematic linter from |
Although /retest |
@cheftako friendly ping. It would be nice to get lint working again against mainline. I would have liked to simply adjust Makefile but I never solved a remaining mystery why it hangs when invoked from kubekins / test-infra. At the same linter version it succeeds very quickly via Github Action. Migrating lint to Github Actions might be superior going forward, even if we did not have that mystery. |
I first tried to reproduce it in the local /github action, but with no luck, I am trying to test it by upgrading golangci-lint to 1.59.0 in #664 |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: cheftako, jkh52 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 |
Update golangci-lint from 1.54.0 to v1.56.2, per needed for go1.22 per golangci/golangci-lint#4273.
Introduce a Github Action based linter.
Fix resulting findings.