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
chore: update golangci-lint to v1.49.0 #977
Conversation
@@ -6,7 +6,6 @@ run: | |||
linters: | |||
disable-all: true | |||
enable: | |||
- deadcode |
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.
WARN [runner] The linter 'varcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
WARN [runner] The linter 'deadcode' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
WARN [runner] The linter 'structcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
Codecov Report
@@ Coverage Diff @@
## master #977 +/- ##
==========================================
- Coverage 59.18% 58.91% -0.28%
==========================================
Files 10 10
Lines 1061 1066 +5
==========================================
Hits 628 628
- Misses 398 403 +5
Partials 35 35 |
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.
LGTM! Pending other review comments.
08d974e
to
5525f2d
Compare
@@ -16,6 +16,10 @@ import ( | |||
"k8s.io/klog/v2" | |||
) | |||
|
|||
const ( | |||
readHeaderTimeout = 5 * time.Second |
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.
should we move this to utils package so it's shared?
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.
If we use constants from utils
package, it'll lead to import cycle, that's why I defined them as local constants.
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
Signed-off-by: Anish Ramasekar <anish.ramasekar@gmail.com>
5525f2d
to
6d8a05a
Compare
Signed-off-by: Anish Ramasekar anish.ramasekar@gmail.com
Reason for Change:
v1.49.0
Requirements
Issue Fixed:
Does this change contain code from or inspired by another project?
If "Yes," did you notify that project's maintainers and provide attribution?
Special Notes for Reviewers: