Migrate golint to v2

```
WARN The configuration comments are not migrated.
WARN Details about the migration: https://golangci-lint.run/product/migration-guide/
WARN The configuration `run.timeout` is ignored. By default, in v2, the timeout is disabled.
```

I've backported comment and timeout

Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>
pull/30752/head
Benoit Tigeot 5 months ago
parent 18ed1cf720
commit d878548168
No known key found for this signature in database
GPG Key ID: 8E6D4FC8AEBDA62C

@ -1,45 +1,63 @@
version: "2"
run: run:
timeout: 10m timeout: 10m
linters: linters:
disable-all: true default: none
enable: enable:
- dupl - dupl
- gofmt
- goimports
- gosimple
- govet - govet
- ineffassign - ineffassign
- misspell - misspell
- nakedret - nakedret
- revive - revive
- unused
- staticcheck - staticcheck
- unused
linters-settings: settings:
gofmt:
simplify: true
goimports:
local-prefixes: helm.sh/helm/v4
dupl: dupl:
threshold: 400 threshold: 400
issues: exclusions:
exclude-rules:
# Helm, and the Go source code itself, sometimes uses these names outside their built-in # Helm, and the Go source code itself, sometimes uses these names outside their built-in
# functions. As the Go source code has re-used these names it's ok for Helm to do the same. # functions. As the Go source code has re-used these names it's ok for Helm to do the same.
# Linting will look for redefinition of built-in id's but we opt-in to the ones we choose to use. # Linting will look for redefinition of built-in id's but we opt-in to the ones we choose to use.
generated: lax
presets:
- comments
- common-false-positives
- legacy
- std-error-handling
rules:
- linters: - linters:
- revive - revive
text: "redefines-builtin-id: redefinition of the built-in function append" text: 'redefines-builtin-id: redefinition of the built-in function append'
- linters: - linters:
- revive - revive
text: "redefines-builtin-id: redefinition of the built-in function clear" text: 'redefines-builtin-id: redefinition of the built-in function clear'
- linters: - linters:
- revive - revive
text: "redefines-builtin-id: redefinition of the built-in function max" text: 'redefines-builtin-id: redefinition of the built-in function max'
- linters: - linters:
- revive - revive
text: "redefines-builtin-id: redefinition of the built-in function min" text: 'redefines-builtin-id: redefinition of the built-in function min'
- linters: - linters:
- revive - revive
text: "redefines-builtin-id: redefinition of the built-in function new" text: 'redefines-builtin-id: redefinition of the built-in function new'
paths:
- third_party$
- builtin$
- examples$
formatters:
enable:
- gofmt
- goimports
settings:
gofmt:
simplify: true
goimports:
local-prefixes:
- helm.sh/helm/v4
exclusions:
generated: lax
paths:
- third_party$
- builtin$
- examples$

Loading…
Cancel
Save