Skip to content

bump golangci-lint to v1.61.0 and cleanup config - #5585

Merged
thaJeztah merged 4 commits into
docker:masterfrom
thaJeztah:bump_golangci_lint
Oct 31, 2024
Merged

bump golangci-lint to v1.61.0 and cleanup config#5585
thaJeztah merged 4 commits into
docker:masterfrom
thaJeztah:bump_golangci_lint

Conversation

@thaJeztah

Copy link
Copy Markdown
Member

  • golangci-lint: enable G204, add #nosec comments instead
  • golangci-lint: move gosec excludes to linters-settings
  • golangci-lint: update comment, and disable "exclude-dirs-use-default"
  • bump golangci-lint to v1.61.0

- A picture of a cute animal (not mandatory but encouraged)

There's only 3 locations where it's hit, so putting #gosec ignore comments
in those locations.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
Also updating a linter that was deprecated;
The linter 'exportloopref' is deprecated (since v1.60.2) due to: Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@codecov-commenter

codecov-commenter commented Oct 30, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 59.64%. Comparing base (aa331e9) to head (9af049c).
Report is 56 commits behind head on master.

Additional details and impacted files
@@ Coverage Diff @@## master #5585 +/- ##
==========================================
- Coverage 59.64% 59.64% -0.01% 
==========================================
Files 346 346 Lines 29207 29208 +1 ==========================================
Hits 17421 17421 - Misses 10817 10818 +1 
Partials 969 969 

Sign up for freeto 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.

3 participants

@thaJeztah@codecov-commenter@vvoland