-
Notifications
You must be signed in to change notification settings - Fork 49
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 frameworks version of golangci-lint #139
Conversation
Use the same linters we use in gatekeeper/ for consistency. Remove usage of deprecated/now-unsupported linters. Make changes to make linting pass. Signed-off-by: Will Beason <willbeason@google.com>
Codecov Report
@@ Coverage Diff @@
## master #139 +/- ##
==========================================
+ Coverage 42.39% 42.52% +0.13%
==========================================
Files 49 49
Lines 2951 2958 +7
==========================================
+ Hits 1251 1258 +7
+ Misses 1321 1320 -1
- Partials 379 380 +1
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Signed-off-by: Will Beason <willbeason@google.com>
Also fix a few typos I found Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
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 with nits
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.
Thank you! LGTM pending @maxsmythe's comments
In some cases I've reworded errors so they do not begin with a proper noun Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
Signed-off-by: Will Beason <willbeason@google.com>
@willbeason good to merge? |
Use the same linters we use in gatekeeper/ for consistency.
Remove usage of deprecated/now-unsupported linters.
Make changes to make linting pass.
Signed-off-by: Will Beason willbeason@google.com