Skip to content

ci: bump golangci-lint to v1.50.1 #580

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

Merged
merged 2 commits into from
Dec 13, 2022
Merged

Conversation

rolinh
Copy link
Member

@rolinh rolinh commented Dec 8, 2022

Also bump the golangci-lint action to v3.3.1. Use the new skip-cache directive to disable caching instead of the more specific options. Remove only-new-issues: false given that false is the default value. Finally, remove skip-go-installation as this is an unknown config option to the linter which gives a warning.

@rolinh rolinh requested a review from willfindlay as a code owner December 8, 2022 08:45
@rolinh rolinh force-pushed the pr/rolinh/ci-bump-golangci-lint branch 2 times, most recently from eb58ea4 to 0b31d33 Compare December 8, 2022 09:32
@rolinh rolinh requested a review from a team as a code owner December 8, 2022 09:32
Copy link
Contributor

@willfindlay willfindlay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you run make format to fix the static check errors? Or if you don't mind me pushing to your branch I can do it.

@rolinh rolinh force-pushed the pr/rolinh/ci-bump-golangci-lint branch from 0b31d33 to cc5a290 Compare December 9, 2022 15:49
@willfindlay
Copy link
Contributor

More struggle, needs a make vendor now

@rolinh rolinh force-pushed the pr/rolinh/ci-bump-golangci-lint branch from cc5a290 to d2d8c82 Compare December 9, 2022 15:55
We now use Go v1.18, there is not reason to use the Go v1.17
compatibility mode anymore.

Signed-off-by: Robin Hahling <[email protected]>
@rolinh rolinh force-pushed the pr/rolinh/ci-bump-golangci-lint branch from d2d8c82 to 7a37f96 Compare December 9, 2022 16:02
The `varcheck` linter has been removed as it is unmaintained.  Also bump
the golangci-lint action to v3.3.1. Use the new `skip-cache` directive
to disable caching instead of the more specific options.  Remove
`only-new-issues: false` given that `false` is the default value.
Finally, remove `skip-go-installation` as this is an unknown config
option to the linter which gives a warning.

Signed-off-by: Robin Hahling <[email protected]>
@rolinh rolinh force-pushed the pr/rolinh/ci-bump-golangci-lint branch from 7a37f96 to b633f81 Compare December 9, 2022 16:08
Copy link
Contributor

@kkourt kkourt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@kkourt kkourt merged commit fddb308 into main Dec 13, 2022
@kkourt kkourt deleted the pr/rolinh/ci-bump-golangci-lint branch December 13, 2022 10:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants