-
Notifications
You must be signed in to change notification settings - Fork 1.8k
ci: use golangci-lint v2 #7029
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: master
Are you sure you want to change the base?
ci: use golangci-lint v2 #7029
Conversation
and use autofix Signed-off-by: Pascal Bourdier <[email protected]>
|
but no DNS entry seems present for |
|
Hi @PascalBourdier Thanks for the contribution, I'll try to review this pr this week. The DNS issue is known, hopefully the website comes back. |
Signed-off-by: Pascal Bourdier <[email protected]>
80079a1 to
a9ee123
Compare
FTR I'd recommend changing this to https://github.com/kudobuilder/kuttl/blob/main/docs/testing/reference.md#testsuite either way, the docs on the website were perpetually out of date, and if the site is restored it will most likely become a redirect. |
|
@porridge Thanks for the ref, I raised the below PR that should at least fix the DNS issue, but there is still likely to be failures in this CI step, since there are over 800 links being checked. |
update golangci-lint to v2 and use autofix
Description of the change:
update golangci-lint to v2
Motivation for the change:
allow golangci-lint v2 features usage
Checklist
If the pull request includes user-facing changes, extra documentation is required:
changelog/fragments(seechangelog/fragments/00-template.yaml)website/content/en/docs