-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy path.golangci.yml
240 lines (233 loc) · 12.1 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
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
205
206
207
208
209
210
211
212
213
214
215
216
217
218
219
220
221
222
223
224
225
226
227
228
229
230
231
232
233
234
235
236
237
238
239
240
run:
tests: true
allow-parallel-runners: true
allow-serial-runners: true
issues:
exclude-generated: strict
exclude-dirs:
- contracts/wasm
- tools/cluster
- documentation
- packages/isc/coreutil
exclude-use-default: false
# Maximum issues count per one linter.
# Set to 0 to disable.
# Default: 50
max-issues-per-linter: 0
# Maximum count of issues with the same text.
# Set to 0 to disable.
# Default: 3
max-same-issues: 0
new-from-rev: ""
exclude:
- 'Error return value of .((os\.)?std(out|err)\..*|.*Close|.*Flush|os\.Remove(All)?|.*print(f|ln)?|os\.(Un)?Setenv). is not checked' # errcheck
- "err113: do not define dynamic errors, use wrapped static errors instead:" # goerr113
- "type name will be used as [0-9A-Za-z_.]+ by other packages, and that stutters; consider calling this" # golint
- "Potential file inclusion via variable" # gosec
- "G404: Use of weak random number generator" # gosec
- "Subprocess launch(ed with variable|ing should be audited)" # gosec
- "Use of unsafe calls should be audited" # gosec
- "G108: Profiling endpoint is automatically exposed on /debug/pprof" # gosec
- "(Expect directory permissions to be 0750 or less|Expect file permissions to be 0600 or less)" # gosec
- "G101: Potential hardcoded credentials" # gosec
- "(G104|G307)" # gosec Duplicated errcheck checks.
- "`[0-9A-Za-z_.]+` - `[0-9A-Za-z_.]+` always receives `[0-9A-Za-z_.]+`" # unparam
- "should have comment .*or be unexported" # revive
- "exported: comment on exported" # revive
- "package-comments" # revive
- "blank-imports" # revive
- "var-naming: don't use leading k in Go names;" # revive
- "var-naming: don't use an underscore in package name" # revive
- "if-return: redundant if ...; err != nil check, just return error instead." # revive
- "unused-parameter: parameter" # revive (maybe we should add this again later?)
- "Error return value of `e.Encode` is not checked" # errcheck (Mostly used in BCS encoding where e.Err() gets used afterwords)
- "may want to add detail/ assignee to this TODO/ FIXME/ BUG comment" # gocritic
exclude-rules:
- path: (_test\.go|example_test\.go|example_[0-9A-Za-z_-]+_test\.go)
linters:
- errcheck
- bodyclose
- noctx
- gosec
- funlen
- gomnd
- path: (_test\.go|example_test\.go|example_[0-9A-Za-z_-]+_test\.go)
linters:
- gocritic
text: "unnecessaryBlock: block doesn't have definitions, can be simply deleted"
- path: packages/isc/coreutil/packages/isc/coreutil/generated_entrypoints.go
linters:
- errcheck
- bodyclose
- noctx
- gosec
- funlen
- gomnd
- gocritic
linters-settings:
revive:
rules:
- name: var-declaration
disabled: true
exclude: [""]
funlen:
lines: 100
statements: 50
gocritic:
enabled-tags:
- diagnostic
- style
- opinionated
disabled-checks:
- paramTypeCombine
- commentedOutCode
- ifElseChain
- commentFormatting
- dupImport
- unnamedResult
- octalLiteral
- whyNoLint
- wrapperFunc
- importShadow
- sloppyReassign
gomnd:
settings:
mnd:
# don't include the "operation" and "assign"
checks: [argument, case, condition, return]
govet:
enable:
- shadow
misspell:
locale: US
nolintlint:
allow-unused: false # report any unused nolint directives
golint:
min-confidence: 0.9
gocyclo:
min-complexity: 20
staticcheck:
checks: ["all"]
stylecheck:
initialisms:
[
"ACL",
"API",
"ASCII",
"CPU",
"CSS",
"DNS",
"EOF",
"GUID",
"HTML",
"HTTP",
"HTTPS",
"ID",
"IP",
"JSON",
"QPS",
"RAM",
"RPC",
"SLA",
"SMTP",
"SQL",
"SSH",
"TCP",
"TLS",
"TTL",
"UDP",
"UI",
"GID",
"UID",
"UUID",
"URI",
"URL",
"UTF8",
"VM",
"XML",
"XMPP",
"XSRF",
"XSS",
"SIP",
"RTP",
"AMQP",
"DB",
"TS",
]
linters:
disable-all: true
enable:
- bodyclose # Checks whether HTTP response body is closed successfully.
- dupl # Tool for code clone detection.
- errcheck # Errcheck is a program for checking for unchecked errors in go programs. These unchecked errors can be critical bugs in some cases
- copyloopvar # Checks for pointers to enclosing loop variables.
- funlen # Tool for detection of long functions.
- gocritic # Provides many diagnostics that check for bugs, performance and style issues.
- gocyclo # Computes and checks the cyclomatic complexity of functions.
- gofmt # Gofmt checks whether code was gofmt-ed. By default this tool runs with -s option to check for code simplification.
- gofumpt # Gofumpt checks whether code was gofumpt-ed. A more strict version of gofmt.
- revive # revive is the newer replacement for golint
- goprintffuncname # Checks that printf-like functions are named with f at the end.
- gosec # Inspects source code for security problems.
- gosimple # Linter for Go source code that specializes in simplifying a code.
- govet # Vet examines Go source code and reports suspicious constructs.
- ineffassign # Detects when assignments to existing variables are not used.
- misspell # Finds commonly misspelled English words in comments.
- nakedret # Finds naked returns in functions greater than a specified function length.
- noctx # noctx finds sending http request without context.Context.
- nolintlint # Reports ill-formed or insufficient nolint directives.
- staticcheck # Staticcheck is a go vet on steroids, applying a ton of static analysis checks.
- unconvert # Remove unnecessary type conversions.
- unparam # Reports unused function parameters.
- unused # Checks Go code for unused constants, variables, functions and types.
- whitespace # Tool for detection of leading and trailing whitespace.
- asasalint # check for pass []any as any in variadic func(...any)
- asciicheck # Simple linter to check that your code does not contain non-ASCII identifiers [fast: true, auto-fix: false]
- bidichk # Checks for dangerous unicode character sequences
- contextcheck # check the function whether use a non-inherited context
- decorder # check declaration order and count of types, constants, variables and functions
- goheader # Checks is file header matches to pattern [fast: true, auto-fix: false]
- gomodguard # Allow and block list linter for direct Go module dependencies. This is different from depguard where there are different block types for example version constraints and module recommendations. [fast: true, auto-fix: false]
- grouper # An analyzer to analyze expression groups.
- importas # Enforces consistent import aliases [fast: true, auto-fix: false]
- predeclared # find code that shadows one of Go's predeclared identifiers [fast: true, auto-fix: false]
- rowserrcheck # checks whether Err of rows is checked successfully [fast: true, auto-fix: false]
- sqlclosecheck # Checks that sql.Rows and sql.Stmt are closed. [fast: true, auto-fix: false]
- tenv # tenv is analyzer that detects using os.Setenv instead of t.Setenv since Go1.17
- tparallel # tparallel detects inappropriate usage of t.Parallel() method in your Go test codes [fast: true, auto-fix: false]
- usestdlibvars # A linter that detect the possibility to use variables/constants from the Go standard library.
- wastedassign # wastedassign finds wasted assignment statements. [fast: true, auto-fix: false]
# - goconst # Finds repeated strings that could be replaced by a constant.
# - depguard # Go linter that checks if package imports are in a list of acceptable packages [fast: true, auto-fix: false]
# nlreturn # nlreturn checks for a new line before return and branch statements to increase code clarity [fast: true, auto-fix: false]
# don't enable:
# gomnd: An analyzer to detect magic numbers.
# cyclop: checks function and package cyclomatic complexity [fast: true, auto-fix: false]
# durationcheck: check for two durations multiplied together [fast: true, auto-fix: false]
# errorlint: go-errorlint is a source code linter for Go software that can be used to find code that will cause problemswith the error wrapping scheme introduced in Go 1.13. [fast: true, auto-fix: false]
# exhaustive: check exhaustiveness of enum switch statements [fast: true, auto-fix: false]
# exhaustivestruct: Checks if all struct's fields are initialized [fast: true, auto-fix: false]
# forbidigo: Forbids identifiers [fast: true, auto-fix: false]
# forcetypeassert: finds forced type assertions [fast: true, auto-fix: false]
# gci: Gci control golang package import order and make it always deterministic. [fast: true, auto-fix: true]
# gochecknoglobals: check that no global variables exist [fast: true, auto-fix: false]
# gochecknoinits: Checks that no init functions are present in Go code [fast: true, auto-fix: false]
# gocognit: Computes and checks the cognitive complexity of functions [fast: true, auto-fix: false]
# godot: Check if comments end in a period [fast: true, auto-fix: true]
# godox: Tool for detection of FIXME, TODO and other comment keywords [fast: true, auto-fix: false]
# gofmt: Gofmt checks whether code was gofmt-ed. By default this tool runs with -s option to check for code simplification [fast: true, auto-fix: true]
# ifshort: Checks that your code uses short syntax for if-statements whenever possible [fast: true, auto-fix: false]
# interfacer: Linter that suggests narrower interface types [fast: true, auto-fix: false]
# lll: Reports long lines [fast: true, auto-fix: false]
# makezero: Finds slice declarations with non-zero initial length [fast: true, auto-fix: false]
# maligned: Tool to detect Go structs that would take less memory if their fields were sorted [fast: true, auto-fix: false]
# nestif: Reports deeply nested if statements [fast: true, auto-fix: false]
# nilerr: Finds the code that returns nil even if it checks that the error is not nil. [fast: true, auto-fix: false]
# paralleltest: paralleltest detects missing usage of t.Parallel() method in your Go test [fast: true, auto-fix: false]
# prealloc: Finds slice declarations that could potentially be preallocated [fast: true, auto-fix: false]
# scopelint: Scopelint checks for unpinned variables in go programs [fast: true, auto-fix: false]
# testpackage: linter that makes you use a separate _test package [fast: true, auto-fix: false]
# thelper: thelper detects golang test helpers without t.Helper() call and checks the consistency of test helpers [fast: true, auto-fix: false]
# typecheck: Like the front-end of a Go compiler, parses and type-checks Go code [fast: true, auto-fix: false]
# wrapcheck: Checks that errors returned from external packages are wrapped [fast: true, auto-fix: false]
# wsl: Whitespace Linter - Forces you to use empty lines! [fast: true, auto-fix: false]