diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-01-19 03:05:47 +0300 |
---|---|---|
committer | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-01-21 23:00:51 +0300 |
commit | 1593037c00cdc7c7ed3407c62b44f4dd25a268e1 (patch) | |
tree | 47248eb2020194b22da316398371a1c906490ef5 /.golangci-strict.yml | |
parent | f48cc807328356526e3fad789e150c5eb87113e6 (diff) |
Revert & amend "disable gochecknoglobals linter"
This reverts & amends commit 0b5a8ea8e (disable gochecknoglobals
linter, 2020-12-17).
We can just exclude the feature variables from the complaining. This
leaves a manageable amount of issues which we can add to "make
lint-strict".
Diffstat (limited to '.golangci-strict.yml')
-rw-r--r-- | .golangci-strict.yml | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/.golangci-strict.yml b/.golangci-strict.yml index 610efbe32..4b784c43d 100644 --- a/.golangci-strict.yml +++ b/.golangci-strict.yml @@ -10,6 +10,7 @@ linters: enable: - errcheck - errorlint + - gochecknoglobals - golint issues: @@ -21,6 +22,10 @@ issues: 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" + - linters: - golint path: "_test.go" text: "context.Context should be the first parameter of a function" |