-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
🐛 upgrade golangci-lint to v1.61.0 #2950
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: sivchari <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: sivchari The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
409f39c
to
469e9d1
Compare
469e9d1
to
539b43b
Compare
Signed-off-by: sivchari <[email protected]>
539b43b
to
c487124
Compare
Signed-off-by: sivchari <[email protected]>
b497534
to
e96da8a
Compare
@@ -1544,7 +1544,7 @@ func CacheTest(createCacheFunc func(config *rest.Config, opts cache.Options) (ca | |||
return obtainedPodNames | |||
}, ConsistOf(tc.expectedPods))) | |||
for _, pod := range obtainedStructuredPodList.Items { | |||
Expect(informer.Get(context.Background(), client.ObjectKeyFromObject(&pod), &pod)).To(Succeed()) //nolint:gosec // We don't retain the pointer | |||
Expect(informer.Get(context.Background(), client.ObjectKeyFromObject(&pod), &pod)).To(Succeed()) // We don't retain the pointer |
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.
Can delete the entire comments for these, the purpose of the comment was to explain the nolint
directive
Upgade the golangci-lint version to v1.61.0.