-
Notifications
You must be signed in to change notification settings - Fork 51
/
.golangci.yml
112 lines (107 loc) · 2.87 KB
/
.golangci.yml
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
run:
tests: true
max-same-issues: 50
output:
print-issued-lines: false
linters:
enable-all: true
disable:
- lll
- typecheck # `go build` catches this, and it doesn't currently work with Go 1.11 modules
- goimports # horrendously slow with go modules :(
- dupl # has never been actually useful
- gochecknoglobals
- gochecknoinits
- funlen
- whitespace
- godox
- wsl
- dogsled
- gomnd
- gocognit
- gocyclo
- godot
- nestif
- testpackage
- gci
- gofumpt
- nlreturn
- cyclop
- paralleltest
- prealloc
- nolintlint
- varnamelen
- nonamedreturns
- nilnil
- ireturn
- exhaustruct
- forcetypeassert
- usestdlibvars
- wrapcheck
- tagliatelle
- makezero
- maintidx
- nilerr
- errchkjson
- mnd # overly sensitive to any number
- tagalign # don't care about struct tag alignment (help:".." tends to be long)
- execinquery
linters-settings:
govet:
use-installed-packages: true
dupl:
threshold: 100
goconst:
min-len: 8
min-occurrences: 3
gocyclo:
min-complexity: 20
gocritic:
disabled-checks:
- ifElseChain
exhaustive:
default-signifies-exhaustive: true
depguard:
rules:
main:
deny:
- pkg: errors
desc: "use github.com/cashapp/hermit/errors"
- pkg: github.com/pkg/errors
desc: "use github.com/cashapp/hermit/errors"
forbidigo:
forbid:
- '^fmt.Errorf'
- '^os.User.*Dir'
issues:
max-per-linter: 0
max-same-issues: 0
exclude-use-default: false
exclude-files:
- cmd/protopkg/main.go
exclude-dirs:
- resources
- old
exclude:
# Captured by errcheck.
- '^(G104|G204):'
# Very commonly not checked.
- 'Error return value of .(.*\.Help|.*\.MarkFlagRequired|(os\.)?std(out|err)\..*|.*Close|.*Flush|os\.Remove(All)?|.*Print(f|ln|)|os\.(Un)?Setenv). is not checked'
# Weird error only seen on Kochiku...
- 'internal error: no range for'
- 'exported method `.*\.(MarshalJSON|UnmarshalJSON|URN|Payload|GoString|Close|Provides|Requires|ExcludeFromHash|MarshalText|UnmarshalText|Description|Check|Poll|Severity)` should have comment or be unexported'
- 'declaration of "err" shadows declaration'
- 'by other packages, and that stutters'
- 'Potential file inclusion via variable'
- 'at least one file in a package should have a package comment'
- 'bad syntax for struct tag pair'
- 'should have.*comment'
- 'uses unkeyed fields'
- 'G307'
- 'strings.Title has been deprecated'
exclude-rules:
# Don't warn on unused parameters.
# Parameter names are useful documentation--even when unused.
# Replacing them with '_' is undesirable.
- linters: [revive]
text: 'unused-parameter: parameter \S+ seems to be unused, consider removing or renaming it as _'