Skip to content
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 to golangci-lint v1.55.0 #4620

Merged
merged 2 commits into from
Oct 24, 2023
Merged

Conversation

thaJeztah
Copy link
Member

@thaJeztah thaJeztah commented Oct 23, 2023

@codecov-commenter
Copy link

codecov-commenter commented Oct 23, 2023

Codecov Report

Merging #4620 (b7b5b31) into master (8bf53ab) will increase coverage by 0.00%.
The diff coverage is 0.00%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4620   +/-   ##
=======================================
  Coverage   59.74%   59.74%           
=======================================
  Files         288      288           
  Lines       24850    24849    -1     
=======================================
  Hits        14846    14846           
+ Misses       9118     9117    -1     
  Partials      886      886           

Latest gosec linter has a regression in parsing "nosec" comments;
see securego/gosec#1046

Signed-off-by: Sebastiaan van Stijn <[email protected]>
@thaJeztah thaJeztah marked this pull request as ready for review October 24, 2023 09:11
@thaJeztah thaJeztah merged commit 39e1de9 into docker:master Oct 24, 2023
74 checks passed
@thaJeztah thaJeztah deleted the bump_golangci_lint branch October 24, 2023 11:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants