mirror of
https://github.com/ceph/ceph-csi.git
synced 2025-03-09 08:59:30 +00:00
f87d06ed85
Several packages are only used while running the e2e suite. These packages are less important to update, as the they can not influence the final executable that is part of the Ceph-CSI container-image. By moving these dependencies out of the main Ceph-CSI go.mod, it is easier to identify if a reported CVE affects Ceph-CSI, or only the testing (like most of the Kubernetes CVEs). Signed-off-by: Niels de Vos <ndevos@ibm.com>
78 lines
2.3 KiB
YAML
78 lines
2.3 KiB
YAML
output:
|
|
# Make output more digestible with quickfix in vim/emacs/etc.
|
|
sort-results: true
|
|
print-issued-lines: false
|
|
|
|
linters:
|
|
# We'll track the golangci-lint default linters manually
|
|
# instead of letting them change without our control.
|
|
disable-all: true
|
|
enable:
|
|
# golangci-lint defaults:
|
|
- errcheck
|
|
- gosimple
|
|
- govet
|
|
- ineffassign
|
|
- staticcheck
|
|
- unused
|
|
|
|
# Our own extras:
|
|
- gofumpt
|
|
- nolintlint # lints nolint directives
|
|
- revive
|
|
|
|
linters-settings:
|
|
govet:
|
|
# These govet checks are disabled by default, but they're useful.
|
|
enable:
|
|
- niliness
|
|
- reflectvaluecompare
|
|
- sortslice
|
|
- unusedwrite
|
|
|
|
errcheck:
|
|
exclude-functions:
|
|
# These methods can not fail.
|
|
# They operate on an in-memory buffer.
|
|
- (*go.uber.org/zap/buffer.Buffer).Write
|
|
- (*go.uber.org/zap/buffer.Buffer).WriteByte
|
|
- (*go.uber.org/zap/buffer.Buffer).WriteString
|
|
|
|
- (*go.uber.org/zap/zapio.Writer).Close
|
|
- (*go.uber.org/zap/zapio.Writer).Sync
|
|
- (*go.uber.org/zap/zapio.Writer).Write
|
|
# Write to zapio.Writer cannot fail,
|
|
# so io.WriteString on it cannot fail.
|
|
- io.WriteString(*go.uber.org/zap/zapio.Writer)
|
|
|
|
# Writing a plain string to a fmt.State cannot fail.
|
|
- io.WriteString(fmt.State)
|
|
|
|
issues:
|
|
# Print all issues reported by all linters.
|
|
max-issues-per-linter: 0
|
|
max-same-issues: 0
|
|
|
|
# Don't ignore some of the issues that golangci-lint considers okay.
|
|
# This includes documenting all exported entities.
|
|
exclude-use-default: false
|
|
|
|
exclude-rules:
|
|
# Don't warn on unused parameters.
|
|
# Parameter names are useful; replacing them with '_' is undesirable.
|
|
- linters: [revive]
|
|
text: 'unused-parameter: parameter \S+ seems to be unused, consider removing or renaming it as _'
|
|
|
|
# staticcheck already has smarter checks for empty blocks.
|
|
# revive's empty-block linter has false positives.
|
|
# For example, as of writing this, the following is not allowed.
|
|
# for foo() { }
|
|
- linters: [revive]
|
|
text: 'empty-block: this block is empty, you can remove it'
|
|
|
|
# Ignore logger.Sync() errcheck failures in example_test.go
|
|
# since those are intended to be uncomplicated examples.
|
|
- linters: [errcheck]
|
|
path: example_test.go
|
|
text: 'Error return value of `logger.Sync` is not checked'
|