Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
rm -rf hack/tools/bin/golangci-lint
What this PR does / why we need it
I wanted to use generics in #55 but the golangci-lint version we are using is too old and gocritic panics.
This commit updates the version to 1.52.2.
Note that "allow-leading-space" is no longer supported as mentioned in golangci/golangci-lint#3063 (comment) and therefore has been removed. All instances of
// nolint
have been update to remove the space.A couple of new warnings were also fixed.
Finally, I have rebase #55 and confirmed we can use generics without linter warnings when using this PR.
Which issue(s) this PR fixes
Fixes # N/A
Describe testing done for PR
make go-lint
I also tried #55 which is introducing generics and ran
make go-lint
. Before this PR, the linter panics but with this PR it all passes.Release note
Additional information
Special notes for your reviewer