|
| 1 | +run: |
| 2 | + timeout: 5m |
| 3 | + |
| 4 | + skip-files: |
| 5 | + - "zz_generated\\..+\\.go$" |
| 6 | + |
| 7 | + issues-exit-code: 1 |
| 8 | + |
| 9 | +output: |
| 10 | + # colored-line-number|line-number|json|tab|checkstyle|code-climate, default is "colored-line-number" |
| 11 | + format: colored-line-number |
| 12 | + |
| 13 | +linters-settings: |
| 14 | + errcheck: |
| 15 | + # report about not checking of errors in type assetions: `a := b.(MyStruct)`; |
| 16 | + # default is false: such cases aren't reported by default. |
| 17 | + check-type-assertions: true |
| 18 | + |
| 19 | + # report about assignment of errors to blank identifier: `num, _ := strconv.Atoi(numStr)`; |
| 20 | + # default is false: such cases aren't reported by default. |
| 21 | + check-blank: true |
| 22 | + |
| 23 | + # [deprecated] comma-separated list of pairs of the form pkg:regex |
| 24 | + # the regex is used to ignore names within pkg. (default "fmt:.*"). |
| 25 | + # see https://github.com/kisielk/errcheck#the-deprecated-method for details |
| 26 | + ignore: fmt:.*,io/ioutil:^Read.* |
| 27 | + |
| 28 | + govet: |
| 29 | + # report about shadowed variables |
| 30 | + check-shadowing: false |
| 31 | + |
| 32 | + golint: |
| 33 | + # minimal confidence for issues, default is 0.8 |
| 34 | + min-confidence: 0.8 |
| 35 | + |
| 36 | + gofmt: |
| 37 | + # simplify code: gofmt with `-s` option, true by default |
| 38 | + simplify: true |
| 39 | + |
| 40 | + goimports: |
| 41 | + # put imports beginning with prefix after 3rd-party packages; |
| 42 | + # it's a comma-separated list of prefixes |
| 43 | + local-prefixes: github.com/crossplane/provider-template |
| 44 | + |
| 45 | + gocyclo: |
| 46 | + # minimal code complexity to report, 30 by default (but we recommend 10-20) |
| 47 | + min-complexity: 15 |
| 48 | + |
| 49 | + cyclop: |
| 50 | + max-complexity: 15 |
| 51 | + |
| 52 | + maligned: |
| 53 | + # print struct with more effective memory layout or not, false by default |
| 54 | + suggest-new: true |
| 55 | + |
| 56 | + dupl: |
| 57 | + # tokens count to trigger issue, 150 by default |
| 58 | + threshold: 100 |
| 59 | + |
| 60 | + goconst: |
| 61 | + # minimal length of string constant, 3 by default |
| 62 | + min-len: 3 |
| 63 | + # minimal occurrences count to trigger, 3 by default |
| 64 | + min-occurrences: 5 |
| 65 | + |
| 66 | + lll: |
| 67 | + # tab width in spaces. Default to 1. |
| 68 | + tab-width: 1 |
| 69 | + |
| 70 | + unused: |
| 71 | + # treat code as a program (not a library) and report unused exported identifiers; default is false. |
| 72 | + # XXX: if you enable this setting, unused will report a lot of false-positives in text editors: |
| 73 | + # if it's called for subdir of a project it can't find funcs usages. All text editor integrations |
| 74 | + # with golangci-lint call it on a directory with the changed file. |
| 75 | + check-exported: false |
| 76 | + |
| 77 | + unparam: |
| 78 | + # Inspect exported functions, default is false. Set to true if no external program/library imports your code. |
| 79 | + # XXX: if you enable this setting, unparam will report a lot of false-positives in text editors: |
| 80 | + # if it's called for subdir of a project it can't find external interfaces. All text editor integrations |
| 81 | + # with golangci-lint call it on a directory with the changed file. |
| 82 | + check-exported: false |
| 83 | + |
| 84 | + prealloc: |
| 85 | + # XXX: we don't recommend using this linter before doing performance profiling. |
| 86 | + # For most programs usage of prealloc will be a premature optimization. |
| 87 | + |
| 88 | + # Report preallocation suggestions only on simple loops that have no returns/breaks/continues/gotos in them. |
| 89 | + # True by default. |
| 90 | + simple: true |
| 91 | + range-loops: true # Report preallocation suggestions on range loops, true by default |
| 92 | + for-loops: false # Report preallocation suggestions on for loops, false by default |
| 93 | + |
| 94 | + gocritic: |
| 95 | + # Enable multiple checks by tags, run `GL_DEBUG=gocritic golangci-lint` run to see all tags and checks. |
| 96 | + # Empty list by default. See https://github.com/go-critic/go-critic#usage -> section "Tags". |
| 97 | + enabled-tags: |
| 98 | + - diagnostic |
| 99 | + - experimental |
| 100 | + - opinionated |
| 101 | + - performance |
| 102 | + - style |
| 103 | + |
| 104 | + # disabled-checks: |
| 105 | + # - unnamedResult |
| 106 | + # - hugeParam |
| 107 | + |
| 108 | + settings: # settings passed to gocritic |
| 109 | + captLocal: # must be valid enabled check name |
| 110 | + paramsOnly: true |
| 111 | + rangeValCopy: |
| 112 | + sizeThreshold: 32 |
| 113 | + |
| 114 | + nolintlint: |
| 115 | + require-explanation: true |
| 116 | + require-specific: true |
| 117 | + |
| 118 | +linters: |
| 119 | + enable: |
| 120 | + - asasalint |
| 121 | + - asciicheck |
| 122 | + - bidichk |
| 123 | + - bodyclose |
| 124 | + - containedctx |
| 125 | + - contextcheck |
| 126 | + - cyclop |
| 127 | + - decorder |
| 128 | + # - depguard |
| 129 | + - dogsled |
| 130 | + - dupl |
| 131 | + - dupword |
| 132 | + - durationcheck |
| 133 | + - errchkjson |
| 134 | + - errname |
| 135 | + - errorlint |
| 136 | + - errcheck |
| 137 | + - exportloopref |
| 138 | + - exhaustive |
| 139 | + - exportloopref |
| 140 | + - forbidigo |
| 141 | + - forcetypeassert |
| 142 | + # - funlen |
| 143 | + # - gci |
| 144 | + - gocheckcompilerdirectives |
| 145 | + - gocognit |
| 146 | + - goconst |
| 147 | + - gocritic |
| 148 | + # - godot |
| 149 | + # - godox |
| 150 | + # - goerr113 |
| 151 | + - gofmt |
| 152 | + - goimports |
| 153 | + - gomnd |
| 154 | + - gocyclo |
| 155 | + - goprintffuncname |
| 156 | + - gosec |
| 157 | + - gosimple |
| 158 | + - govet |
| 159 | + - ineffassign |
| 160 | + - loggercheck |
| 161 | + - maintidx |
| 162 | + - makezero |
| 163 | + - misspell |
| 164 | + - nestif |
| 165 | + - nilerr |
| 166 | + - nilnil |
| 167 | + - nlreturn |
| 168 | + - noctx |
| 169 | + - nolintlint |
| 170 | + - paralleltest |
| 171 | + - prealloc |
| 172 | + - predeclared |
| 173 | + - reassign |
| 174 | + # - revive |
| 175 | + - staticcheck |
| 176 | + # - stylecheck |
| 177 | + - tenv |
| 178 | + - thelper |
| 179 | + - typecheck |
| 180 | + - unconvert |
| 181 | + - unparam |
| 182 | + - unused |
| 183 | + - usestdlibvars |
| 184 | + - varnamelen |
| 185 | + - whitespace |
| 186 | + # - wrapcheck |
| 187 | + |
| 188 | + presets: |
| 189 | + - bugs |
| 190 | + - unused |
| 191 | + fast: false |
| 192 | + |
| 193 | + |
| 194 | +issues: |
| 195 | + # Excluding configuration per-path and per-linter |
| 196 | + exclude-rules: |
| 197 | + # Exclude some linters from running on tests files. |
| 198 | + - path: _test(ing)?\.go |
| 199 | + linters: |
| 200 | + - gocyclo |
| 201 | + - errcheck |
| 202 | + - dupl |
| 203 | + - gosec |
| 204 | + - exportloopref |
| 205 | + - unparam |
| 206 | + |
| 207 | + # Ease some gocritic warnings on test files. |
| 208 | + - path: _test\.go |
| 209 | + text: "(unnamedResult|exitAfterDefer)" |
| 210 | + linters: |
| 211 | + - gocritic |
| 212 | + |
| 213 | + # These are performance optimisations rather than style issues per se. |
| 214 | + # They warn when function arguments or range values copy a lot of memory |
| 215 | + # rather than using a pointer. |
| 216 | + - text: "(hugeParam|rangeValCopy):" |
| 217 | + linters: |
| 218 | + - gocritic |
| 219 | + |
| 220 | + # This "TestMain should call os.Exit to set exit code" warning is not clever |
| 221 | + # enough to notice that we call a helper method that calls os.Exit. |
| 222 | + - text: "SA3000:" |
| 223 | + linters: |
| 224 | + - staticcheck |
| 225 | + |
| 226 | + - text: "k8s.io/api/core/v1" |
| 227 | + linters: |
| 228 | + - goimports |
| 229 | + |
| 230 | + # This is a "potential hardcoded credentials" warning. It's triggered by |
| 231 | + # any variable with 'secret' in the same, and thus hits a lot of false |
| 232 | + # positives in Kubernetes land where a Secret is an object type. |
| 233 | + - text: "G101:" |
| 234 | + linters: |
| 235 | + - gosec |
| 236 | + - gas |
| 237 | + |
| 238 | + # This is an 'errors unhandled' warning that duplicates errcheck. |
| 239 | + - text: "G104:" |
| 240 | + linters: |
| 241 | + - gosec |
| 242 | + - gas |
| 243 | + |
| 244 | + # Independently from option `exclude` we use default exclude patterns, |
| 245 | + # it can be disabled by this option. To list all |
| 246 | + # excluded by default patterns execute `golangci-lint run --help`. |
| 247 | + # Default value for this option is true. |
| 248 | + exclude-use-default: false |
| 249 | + |
| 250 | + # Show only new issues: if there are unstaged changes or untracked files, |
| 251 | + # only those changes are analyzed, else only changes in HEAD~ are analyzed. |
| 252 | + # It's a super-useful option for integration of golangci-lint into existing |
| 253 | + # large codebase. It's not practical to fix all existing issues at the moment |
| 254 | + # of integration: much better don't allow issues in new code. |
| 255 | + # Default is false. |
| 256 | + new: false |
| 257 | + |
| 258 | + # Maximum issues count per one linter. Set to 0 to disable. Default is 50. |
| 259 | + max-per-linter: 0 |
| 260 | + |
| 261 | + # Maximum count of issues with the same text. Set to 0 to disable. Default is 3. |
| 262 | + max-same-issues: 0 |
0 commit comments