Skip to content

Commit

Permalink
chore: disable deprecated linters
Browse files Browse the repository at this point in the history
  • Loading branch information
grdryn committed Jan 8, 2025
1 parent 2d7c4fa commit a4cd34c
Showing 1 changed file with 4 additions and 2 deletions.
6 changes: 4 additions & 2 deletions .golangci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -97,11 +97,13 @@ linters:
- varnamelen # doesnot allow shorter names like c,k etc. But golang prefers short named vars.
- wsl # [too strict and mostly code is not more readable] whitespace linter forces you to use empty lines
- wrapcheck # check if this is required. Prevents direct return of err.
- exportloopref # Since Go1.22 (loopvar) this linter is no longer relevant. Replaced by copyloopvar.
- execinquery # Deprecated upstream. Not applicable since we don't make DB queries.

# Need to check
- nlreturn # [too strict and mostly code is not more readable] checks for a new line before return and branch statements to increase code clarity
- err113 # [too strict] checks the errors handling expressions

# To be fixed
- gocognit # https://github.com/opendatahub-io/opendatahub-operator/issues/709
- cyclop # https://github.com/opendatahub-io/opendatahub-operator/issues/709
Expand All @@ -115,4 +117,4 @@ issues:
- path: tests/*/(.+)_test\.go
linters:
- typecheck
- dupl
- dupl

0 comments on commit a4cd34c

Please sign in to comment.