diff options
author | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-01-19 03:25:00 +0300 |
---|---|---|
committer | Ævar Arnfjörð Bjarmason <avarab@gmail.com> | 2021-01-21 23:00:51 +0300 |
commit | e1dc9268fd200fdc8aeaf87161465ee8c31c672c (patch) | |
tree | b8282291dd166224b1153397dc410c5eb0c190a6 /.golangci-strict.yml | |
parent | 1593037c00cdc7c7ed3407c62b44f4dd25a268e1 (diff) |
Lint: turn "golint" on by default
Since we've got quite a common warning about things needing comments,
add a gigantic list of those exclusions. Generated with:
make lint |grep golint | sort
| perl -pe 's/^(.*?):\d+:\d\s*golint\s*(.*)/ - linters: \n - golint\n path: "$1"\n text: "$2"/g'
Except I then had to escape "(" in the text to "\\(" because they're
regexes. I suggested simply turning this specific warning off, but
others in today's team meeting wanted to keep it. So here it is as a
giant list.
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 4b784c43d..e75deddf9 100644 --- a/.golangci-strict.yml +++ b/.golangci-strict.yml @@ -11,7 +11,6 @@ linters: - errcheck - errorlint - gochecknoglobals - - golint issues: exclude-use-default: false @@ -25,10 +24,6 @@ issues: - 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" # Maximum issues count per one linter. Set to 0 to disable. Default is 50. max-issues-per-linter: 0 # Maximum count of issues with the same text. Set to 0 to disable. Default is 3. |