-
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
Fix gosec overflow alerts #5799
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #5799 +/- ##
=====================================
Coverage 84.5% 84.5%
=====================================
Files 272 272
Lines 22771 22776 +5
=====================================
+ Hits 19251 19256 +5
Misses 3177 3177
Partials 343 343 |
9d84ad7
to
b21f2f9
Compare
This is missing fixes for all the other lint errors:
|
7f174b5
to
5880fee
Compare
Duh you're right, my bad. I forgot that the linter stops after the first package with failures. I have fixed everything else. |
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.com>
Co-authored-by: Tyler Yahn <MrAlias@users.noreply.github.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.
👍
To allow the golangci-lint upgrade in #5796.