|
1 | | -run: |
2 | | - timeout: 2m |
3 | | - |
4 | | -linters-settings: |
5 | | - errcheck: |
6 | | - # report about not checking of errors in type assetions: `a := b.(MyStruct)`; |
7 | | - # default is false: such cases aren't reported by default. |
8 | | - check-type-assertions: false |
9 | | - |
10 | | - # report about assignment of errors to blank identifier: `num, _ := strconv.Atoi(numStr)`; |
11 | | - # default is false: such cases aren't reported by default. |
12 | | - check-blank: false |
13 | | - |
14 | | - exclude-functions: |
15 | | - - fmt:.*,io |
16 | | - - ioutil:^Read.* |
17 | | - |
18 | | - govet: |
19 | | - disable: |
20 | | - - shadow |
21 | | - |
22 | | - gofmt: |
23 | | - # simplify code: gofmt with `-s` option, true by default |
24 | | - simplify: true |
25 | | - |
26 | | - goimports: |
27 | | - # put imports beginning with prefix after 3rd-party packages; |
28 | | - # it's a comma-separated list of prefixes |
29 | | - local-prefixes: github.com/SAP/crossplane-provider-cloudfoundry |
30 | | - |
31 | | - gocyclo: |
32 | | - # minimal code complexity to report, 30 by default (but we recommend 10-20) |
33 | | - min-complexity: 10 |
34 | | - |
35 | | - dupl: |
36 | | - # tokens count to trigger issue, 150 by default |
37 | | - threshold: 100 |
38 | | - |
39 | | - goconst: |
40 | | - # minimal length of string constant, 3 by default |
41 | | - min-len: 3 |
42 | | - # minimal occurrences count to trigger, 3 by default |
43 | | - min-occurrences: 5 |
44 | | - |
45 | | - lll: |
46 | | - # tab width in spaces. Default to 1. |
47 | | - tab-width: 1 |
48 | | - |
49 | | - unparam: |
50 | | - # Inspect exported functions, default is false. Set to true if no external program/library imports your code. |
51 | | - # XXX: if you enable this setting, unparam will report a lot of false-positives in text editors: |
52 | | - # if it's called for subdir of a project it can't find external interfaces. All text editor integrations |
53 | | - # with golangci-lint call it on a directory with the changed file. |
54 | | - check-exported: false |
55 | | - |
56 | | - nakedret: |
57 | | - # make an issue if func has more lines of code than this setting and it has naked returns; default is 30 |
58 | | - max-func-lines: 30 |
59 | | - |
60 | | - prealloc: |
61 | | - # XXX: we don't recommend using this linter before doing performance profiling. |
62 | | - # For most programs usage of prealloc will be a premature optimization. |
63 | | - |
64 | | - # Report preallocation suggestions only on simple loops that have no returns/breaks/continues/gotos in them. |
65 | | - # True by default. |
66 | | - simple: true |
67 | | - range-loops: true # Report preallocation suggestions on range loops, true by default |
68 | | - for-loops: false # Report preallocation suggestions on for loops, false by default |
69 | | - |
70 | | - gocritic: |
71 | | - # Enable multiple checks by tags, run `GL_DEBUG=gocritic golangci-lint` run to see all tags and checks. |
72 | | - # Empty list by default. See https://github.com/go-critic/go-critic#usage -> section "Tags". |
73 | | - enabled-tags: |
74 | | - - performance |
75 | | - |
76 | | - settings: # settings passed to gocritic |
77 | | - captLocal: # must be valid enabled check name |
78 | | - paramsOnly: true |
79 | | - rangeValCopy: |
80 | | - sizeThreshold: 32 |
81 | | - |
| 1 | +version: "2" |
82 | 2 | linters: |
83 | 3 | enable: |
| 4 | + - asasalint |
| 5 | + - asciicheck |
| 6 | + - bidichk |
| 7 | + - bodyclose |
| 8 | + - contextcheck |
| 9 | + - durationcheck |
| 10 | + - errchkjson |
| 11 | + - errorlint |
| 12 | + - exhaustive |
| 13 | + - gocheckcompilerdirectives |
| 14 | + - gochecksumtype |
84 | 15 | - goconst |
85 | 16 | - gocritic |
86 | 17 | - gocyclo |
87 | | - - gofmt # We enable this as well as goimports for its simplify mode. |
88 | | - - goimports |
89 | | - - gosimple |
90 | | - - govet |
| 18 | + - gosec |
| 19 | + - gosmopolitan |
| 20 | + - loggercheck |
| 21 | + - makezero |
91 | 22 | - misspell |
| 23 | + - musttag |
92 | 24 | - nakedret |
| 25 | + - nilerr |
| 26 | + - nilnesserr |
| 27 | + - noctx |
93 | 28 | - prealloc |
94 | | - - staticcheck |
| 29 | + - protogetter |
| 30 | + - reassign |
| 31 | + - recvcheck |
| 32 | + - rowserrcheck |
| 33 | + - spancheck |
| 34 | + - sqlclosecheck |
| 35 | + - testifylint |
95 | 36 | - unconvert |
96 | | - - unused |
97 | | - |
98 | | - presets: |
99 | | - - bugs |
100 | | - - unused |
101 | | - fast: false |
102 | | - |
103 | | - |
| 37 | + - unparam |
| 38 | + - zerologlint |
| 39 | + settings: |
| 40 | + dupl: |
| 41 | + threshold: 100 |
| 42 | + errcheck: |
| 43 | + check-type-assertions: false |
| 44 | + check-blank: false |
| 45 | + goconst: |
| 46 | + min-len: 3 |
| 47 | + min-occurrences: 5 |
| 48 | + gocritic: |
| 49 | + enabled-tags: |
| 50 | + - performance |
| 51 | + settings: |
| 52 | + captLocal: |
| 53 | + paramsOnly: true |
| 54 | + rangeValCopy: |
| 55 | + sizeThreshold: 32 |
| 56 | + gocyclo: |
| 57 | + min-complexity: 10 |
| 58 | + lll: |
| 59 | + tab-width: 1 |
| 60 | + nakedret: |
| 61 | + max-func-lines: 30 |
| 62 | + prealloc: |
| 63 | + simple: true |
| 64 | + range-loops: true |
| 65 | + for-loops: false |
| 66 | + unparam: |
| 67 | + check-exported: false |
| 68 | + exclusions: |
| 69 | + generated: lax |
| 70 | + rules: |
| 71 | + - linters: |
| 72 | + - dupl |
| 73 | + - errcheck |
| 74 | + - exportloopref |
| 75 | + - gocyclo |
| 76 | + - gosec |
| 77 | + - unparam |
| 78 | + path: _test(ing)?\.go |
| 79 | + - linters: |
| 80 | + - gocritic |
| 81 | + path: _test\.go |
| 82 | + text: (unnamedResult|exitAfterDefer) |
| 83 | + - linters: |
| 84 | + - gocritic |
| 85 | + text: '(hugeParam|rangeValCopy):' |
| 86 | + - linters: |
| 87 | + - staticcheck |
| 88 | + text: 'SA3000:' |
| 89 | + - linters: |
| 90 | + - gosec |
| 91 | + text: 'G101:' |
| 92 | + - linters: |
| 93 | + - gosec |
| 94 | + text: 'G104:' |
| 95 | + paths: |
| 96 | + - zz_generated\..+\.go$ |
| 97 | + - third_party$ |
| 98 | + - builtin$ |
| 99 | + - examples$ |
104 | 100 | issues: |
105 | | - # Excluding configuration per-path and per-linter |
106 | | - exclude-rules: |
107 | | - # Exclude some linters from running on tests files. |
108 | | - - path: _test(ing)?\.go |
109 | | - linters: |
110 | | - - gocyclo |
111 | | - - errcheck |
112 | | - - dupl |
113 | | - - gosec |
114 | | - - exportloopref |
115 | | - - unparam |
116 | | - |
117 | | - # Ease some gocritic warnings on test files. |
118 | | - - path: _test\.go |
119 | | - text: "(unnamedResult|exitAfterDefer)" |
120 | | - linters: |
121 | | - - gocritic |
122 | | - |
123 | | - # These are performance optimisations rather than style issues per se. |
124 | | - # They warn when function arguments or range values copy a lot of memory |
125 | | - # rather than using a pointer. |
126 | | - - text: "(hugeParam|rangeValCopy):" |
127 | | - linters: |
128 | | - - gocritic |
129 | | - |
130 | | - # This "TestMain should call os.Exit to set exit code" warning is not clever |
131 | | - # enough to notice that we call a helper method that calls os.Exit. |
132 | | - - text: "SA3000:" |
133 | | - linters: |
134 | | - - staticcheck |
135 | | - |
136 | | - - text: "k8s.io/api/core/v1" |
137 | | - linters: |
138 | | - - goimports |
139 | | - |
140 | | - # This is a "potential hardcoded credentials" warning. It's triggered by |
141 | | - # any variable with 'secret' in the same, and thus hits a lot of false |
142 | | - # positives in Kubernetes land where a Secret is an object type. |
143 | | - - text: "G101:" |
144 | | - linters: |
145 | | - - gosec |
146 | | - - gas |
147 | | - |
148 | | - # This is an 'errors unhandled' warning that duplicates errcheck. |
149 | | - - text: "G104:" |
150 | | - linters: |
151 | | - - gosec |
152 | | - - gas |
153 | | - |
154 | | - # Independently from option `exclude` we use default exclude patterns, |
155 | | - # it can be disabled by this option. To list all |
156 | | - # excluded by default patterns execute `golangci-lint run --help`. |
157 | | - # Default value for this option is true. |
158 | | - exclude-use-default: false |
159 | | - |
160 | | - # Show only new issues: if there are unstaged changes or untracked files, |
161 | | - # only those changes are analyzed, else only changes in HEAD~ are analyzed. |
162 | | - # It's a super-useful option for integration of golangci-lint into existing |
163 | | - # large codebase. It's not practical to fix all existing issues at the moment |
164 | | - # of integration: much better don't allow issues in new code. |
165 | | - # Default is false. |
166 | | - new: false |
167 | | - |
168 | | - # Maximum count of issues with the same text. Set to 0 to disable. Default is 3. |
169 | 101 | max-same-issues: 0 |
170 | | - |
171 | | - exclude-files: |
172 | | - - "zz_generated\\..+\\.go$" |
| 102 | + new: false |
| 103 | +formatters: |
| 104 | + enable: |
| 105 | + - gofmt |
| 106 | + - goimports |
| 107 | + settings: |
| 108 | + gofmt: |
| 109 | + simplify: true |
| 110 | + goimports: |
| 111 | + local-prefixes: |
| 112 | + - github.com/SAP/crossplane-provider-cloudfoundry |
| 113 | + exclusions: |
| 114 | + generated: lax |
| 115 | + paths: |
| 116 | + - zz_generated\..+\.go$ |
| 117 | + - third_party$ |
| 118 | + - builtin$ |
| 119 | + - examples$ |
0 commit comments