-
-
Notifications
You must be signed in to change notification settings - Fork 1.8k
/
.golangci.yaml
74 lines (71 loc) · 2.52 KB
/
.golangci.yaml
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
# TODO: enable this when we have coverage on docstring comments
#issues:
# # The list of ids of default excludes to include or disable.
# include:
# - EXC0002 # disable excluding of issues about comments from golint
linters-settings:
funlen:
# Checks the number of lines in a function.
# If lower than 0, disable the check.
# Default: 60
# TODO: drop this down over time...
lines: 110
# Checks the number of statements in a function.
# If lower than 0, disable the check.
# Default: 40
statements: 60
# TODO: use the default linters for now, but include these over time
#linters:
# # inverted configuration with `enable-all` and `disable` is not scalable during updates of golangci-lint
# disable-all: true
# enable:
# - asciicheck
# - bodyclose
# - depguard
# - dogsled
# - dupl
# - errcheck
# - exportloopref
# - funlen
# - gocognit
# - goconst
# - gocritic
# - gocyclo
# - gofmt
# - goimports
# - goprintffuncname
# - gosec
# - gosimple
# - govet
# - ineffassign
# - misspell
# - nakedret
# - nolintlint
# - revive
# - staticcheck
# - stylecheck
# - typecheck
# - unconvert
# - unparam
# - unused
# - whitespace
# do not enable...
# - gochecknoglobals
# - gochecknoinits # this is too aggressive
# - godot
# - godox
# - goerr113
# - golint # deprecated
# - gomnd # this is too aggressive
# - interfacer # this is a good idea, but is no longer supported and is prone to false positives
# - lll # without a way to specify per-line exception cases, this is not usable
# - maligned # this is an excellent linter, but tricky to optimize and we are not sensitive to memory layout optimizations
# - nestif
# - prealloc # following this rule isn't consistently a good idea, as it sometimes forces unnecessary allocations that result in less idiomatic code
# - scopelint # deprecated
# - testpackage
# - wsl # this doens't have an auto-fixer yet and is pretty noisy (https://github.com/bombsimon/wsl/issues/90)
# - varcheck # deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
# - deadcode # deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
# - structcheck # deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
# - rowserrcheck # we're not using sql.Rows at all in the codebase