f9c80be1bb
Remove nolint-comments that weren't hit by linters, and remove the "structcheck"
and "varcheck" linters, as they have been deprecated:
WARN [runner] The linter 'structcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
WARN [runner] The linter 'varcheck' is deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
WARN [linters context] structcheck is disabled because of generics. You can track the evolution of the generics support by following the https://github.com/golangci/golangci-lint/issues/2649.
Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
56 lines
1.1 KiB
YAML
56 lines
1.1 KiB
YAML
linters:
|
|
enable:
|
|
- exportloopref # Checks for pointers to enclosing loop variables
|
|
- gofmt
|
|
- goimports
|
|
- gosec
|
|
- ineffassign
|
|
- misspell
|
|
- nolintlint
|
|
- revive
|
|
- staticcheck
|
|
- tenv # Detects using os.Setenv instead of t.Setenv since Go 1.17
|
|
- unconvert
|
|
- unused
|
|
- vet
|
|
disable:
|
|
- errcheck
|
|
|
|
issues:
|
|
include:
|
|
- EXC0002
|
|
max-issues-per-linter: 0
|
|
max-same-issues: 0
|
|
|
|
# Only using / doesn't work due to https://github.com/golangci/golangci-lint/issues/1398.
|
|
exclude-rules:
|
|
- path: 'cmd[\\/]containerd[\\/]builtins[\\/]'
|
|
text: "blank-imports:"
|
|
- path: 'contrib[\\/]fuzz[\\/]'
|
|
text: "exported: func name will be used as fuzz.Fuzz"
|
|
|
|
linters-settings:
|
|
gosec:
|
|
# The following issues surfaced when `gosec` linter
|
|
# was enabled. They are temporarily excluded to unblock
|
|
# the existing workflow, but still to be addressed by
|
|
# by future works.
|
|
excludes:
|
|
- G204
|
|
- G305
|
|
- G306
|
|
- G402
|
|
- G404
|
|
|
|
run:
|
|
timeout: 8m
|
|
skip-dirs:
|
|
- api
|
|
- cluster
|
|
- design
|
|
- docs
|
|
- docs/man
|
|
- releases
|
|
- reports
|
|
- test # e2e scripts
|