-
Notifications
You must be signed in to change notification settings - Fork 246
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
Update Go to 1.19.3, golangci-lint to 1.50.1 #810
Conversation
This fixes the following contextcheck linter error: Function `newIOReaderClient` should pass the context parameter (contextcheck) Signed-off-by: Tobias Klauser <tobias@cilium.io>
2e96b3c
to
203a7d8
Compare
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.
Awesome, thank you for taking care of this!
Seems the linter is a bit more sensitive towards package doc comments? https://github.com/cilium/hubble/actions/runs/3420125081/jobs/5694532263 |
This will re-enable the contextcheck linter for Go ≥ 1.18. Update Go compatibility for the gosimple linter to 1.19. This should have been done in commit 0e85e38 ("Update Go to 1.19"). Remove the deadcode, structcheck and varcheck linters that have been deprecated since golangci-lint v1.49.0. Disable the revive linter warnings for missing package level godoc comments. Signed-off-by: Tobias Klauser <tobias@cilium.io>
Signed-off-by: Tobias Klauser <tobias@cilium.io>
203a7d8
to
08067b0
Compare
"Fixed" by ignoring the warning about missing package-level godocs for now. I'll send a followup PR to add all missing godoc comments. |
See commits for details.