Skip to content

Commit

Permalink
Remove old linters and config workaround
Browse files Browse the repository at this point in the history
  • Loading branch information
pkwarren committed May 28, 2024
1 parent 8fa1190 commit 28d3af0
Showing 1 changed file with 0 additions and 13 deletions.
13 changes: 0 additions & 13 deletions .golangci.yml
Original file line number Diff line number Diff line change
@@ -1,9 +1,6 @@
linters-settings:
errcheck:
check-type-assertions: true
# https://github.com/golangci/golangci-lint/pull/4734.
# Remove when upgrading to next golangci-lint release.
ignore: ""
forbidigo:
forbid:
- '^fmt\.Print'
Expand All @@ -30,31 +27,21 @@ linters:
enable-all: true
disable:
- cyclop # covered by gocyclo
- deadcode # abandoned
- depguard # newer versions require explicit config to depend on anything outside of the Go stdlib
- execinquery # deprecated by author
- exhaustivestruct # replaced by exhaustruct
- funlen # rely on code review to limit function length
- gocognit # dubious "cognitive overhead" quantification
- gofumpt # prefer standard gofmt
- goimports # rely on gci instead
- golint # deprecated by Go team
- gomnd # replaced by mnd
- ifshort # deprecated by author
- inamedparam # convention is not followed
- interfacer # deprecated by author
- ireturn # "accept interfaces, return structs" isn't ironclad
- lll # don't want hard limits for line length
- maintidx # covered by gocyclo
- maligned # readability trumps efficient struct packing
- mnd # some unnamed constants are okay
- nlreturn # generous whitespace violates house style
- nonamedreturns # named returns are fine; it's *bare* returns that are bad
- nosnakecase # deprecated in https://github.com/golangci/golangci-lint/pull/3065
- scopelint # deprecated by author
- structcheck # abandoned
- testpackage # internal tests are fine
- varcheck # abandoned
- wrapcheck # don't _always_ need to wrap errors
- wsl # generous whitespace violates house style
issues:
Expand Down

0 comments on commit 28d3af0

Please sign in to comment.