summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorValentin Rothberg <rothberg@redhat.com>2020-01-13 12:03:57 +0100
committerValentin Rothberg <rothberg@redhat.com>2020-01-13 14:27:02 +0100
commit270d892c3d77de4fd8e6341193175c0572fb5f99 (patch)
tree2f48c0dcd18d0b5dca52569eb523ba4d5790519f
parenteeb61427bb67db0c385df8073eab2acdd18ab503 (diff)
downloadpodman-270d892c3d77de4fd8e6341193175c0572fb5f99.tar.gz
podman-270d892c3d77de4fd8e6341193175c0572fb5f99.tar.bz2
podman-270d892c3d77de4fd8e6341193175c0572fb5f99.zip
linter: blacklist linters instead of whitelisting
Blacklist linters instead of whitelisting them. This way, we will benefit from new linters when updating and it's easier to actually find and fix open issues. Signed-off-by: Valentin Rothberg <rothberg@redhat.com>
-rw-r--r--.golangci.yml53
1 files changed, 17 insertions, 36 deletions
diff --git a/.golangci.yml b/.golangci.yml
index bbf00aa05..bec7b495d 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -17,39 +17,20 @@ run:
skip-files:
- iopodman.go
linters:
- disable-all: true
- enable:
- # - bodyclose
- - deadcode
- - depguard
- # dupl really overdid it; disabling
- # - dupl
- - errcheck
- - gofmt
- - gosimple
- - govet
- - ineffassign
- - nakedret
- - staticcheck
- - structcheck
- - typecheck
- - unused
- - varcheck
- # - gochecknoglobals
- # - gochecknoinits
- # - goconst
- # - gocritic
- # - gocyclo
- # - goimports
- # - golint
- # - gosec
- - interfacer
- # - lll
- # - maligned
- # - misspell
- # - prealloc
- - scopelint
- - stylecheck
- - unconvert
- # I think we should uncomment this one and used it
- # - unparam
+ enable-all: true
+ disable:
+ - dupl
+ - funlen
+ - gochecknoglobals
+ - gochecknoinits
+ - goconst
+ - gocritic
+ - gocyclo
+ - golint
+ - goimports
+ - gosec
+ - lll
+ - maligned
+ - misspell
+ - prealloc
+ - unparam