diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-01-19 04:05:00 +0300 |
---|---|---|
committer | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-01-21 23:00:51 +0300 |
commit | 6e7eea01e3c65929ff20ac6f3eeba2c2636b996f (patch) | |
tree | 2ec7b9136dca8834dfae57eb1647159cf17c2414 /.golangci-strict.yml | |
parent | e1dc9268fd200fdc8aeaf87161465ee8c31c672c (diff) |
Lint: turn "errcheck" on by default
We can turn this on by default by being a bit more permissive about
common cases in non-*_test.go files that should be OK not to check the
return value of most of the time (e.g. os.Remove()), and then having a
giant exclusions list for the rest.
Diffstat (limited to '.golangci-strict.yml')
-rw-r--r-- | .golangci-strict.yml | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/.golangci-strict.yml b/.golangci-strict.yml index e75deddf9..05daa7c0c 100644 --- a/.golangci-strict.yml +++ b/.golangci-strict.yml @@ -8,7 +8,6 @@ run: linters: disable-all: true enable: - - errcheck - errorlint - gochecknoglobals @@ -17,10 +16,6 @@ issues: # Excluding configuration per-path, per-linter, per-text and per-source exclude-rules: - linters: - - errcheck - path: "_test.go" - text: "Error return value of `(cc|con(ns?)?|db|closer|lis(tener)?).Close` is not checked" - - linters: - gochecknoglobals path: "internal/metadata/featureflag/feature_flags.go" text: "is a global variable" |