mirror of
https://github.com/ceph/ceph-csi.git
synced 2024-11-25 07:40:19 +00:00
0f2daca5c2
The new release has some important fixes available with it Ref: https://github.com/ThalesGroup/kmip-go/releases/tag/v0.0.8 Signed-off-by: Humble Chirammal <hchiramm@redhat.com>
233 lines
7.4 KiB
YAML
233 lines
7.4 KiB
YAML
# This file contains all available configuration options
|
|
# with their default values.
|
|
|
|
# options for analysis running
|
|
run:
|
|
tests: true
|
|
skip-files:
|
|
- requests.go
|
|
|
|
# output configuration options
|
|
output:
|
|
# colored-line-number|line-number|json|tab|checkstyle|code-climate, default is "colored-line-number"
|
|
format: colored-line-number
|
|
|
|
# print lines of code with issue, default is true
|
|
print-issued-lines: true
|
|
|
|
# print linter name in the end of issue text, default is true
|
|
print-linter-name: true
|
|
|
|
|
|
# all available settings of specific linters
|
|
linters-settings:
|
|
govet:
|
|
# report about shadowed variables
|
|
check-shadowing: false
|
|
dupl:
|
|
# tokens count to trigger issue, 150 by default
|
|
threshold: 100
|
|
exhaustive:
|
|
default-signifies-exhaustive: true
|
|
goconst:
|
|
# minimal length of string constant, 3 by default
|
|
min-len: 3
|
|
# minimal occurrences count to trigger, 3 by default
|
|
min-occurrences: 3
|
|
depguard:
|
|
list-type: blacklist
|
|
include-go-root: false
|
|
packages:
|
|
- github.com/magiconair/properties/assert
|
|
- gopkg.in/go-playground/assert.v1
|
|
- github.com/pborman/uuid #replace with github.com/google/uuid
|
|
inTests:
|
|
- github.com/davecgh/go-spew/spew
|
|
- github.com/stretchr/testify
|
|
misspell:
|
|
# Correct spellings using locale preferences for US or UK.
|
|
# Default is to use a neutral variety of English.
|
|
# Setting locale to US will correct the British spelling of 'colour' to 'color'.
|
|
locale: US
|
|
unused:
|
|
# treat code as a program (not a library) and report unused exported identifiers; default is false.
|
|
# XXX: if you enable this setting, unused will report a lot of false-positives in text editors:
|
|
# if it's called for subdir of a project it can't find funcs usages. All text editor integrations
|
|
# with golangci-lint call it on a directory with the changed file.
|
|
check-exported: false
|
|
gocritic:
|
|
# Which checks should be disabled; can't be combined with 'enabled-checks'; default is empty
|
|
disabled-checks:
|
|
revive:
|
|
ignore-generated-header: true
|
|
wsl:
|
|
allow-cuddle-declarations: true
|
|
allow-separated-leading-comment: true
|
|
allow-assign-and-anything: true
|
|
|
|
linters:
|
|
# to try out individual linters: golangci-lint run -E gocyclo,gosimple
|
|
enable:
|
|
# default linters
|
|
- staticcheck
|
|
- deadcode
|
|
- errcheck
|
|
- gosimple
|
|
- govet
|
|
- ineffassign
|
|
- structcheck
|
|
- unused
|
|
- varcheck
|
|
|
|
# additional linters
|
|
- asciicheck
|
|
- bidichk
|
|
## - bodyclose # its all false positives with requester and sling, which both close the body already
|
|
- containedctx
|
|
- contextcheck
|
|
# - cyclop # need to analyze findings
|
|
- decorder
|
|
- depguard
|
|
## - dogsled # checks for too many blank identifiers. don't care
|
|
- dupl
|
|
- durationcheck
|
|
- errchkjson
|
|
- errname
|
|
- errorlint
|
|
- exhaustive
|
|
- exportloopref
|
|
- forbidigo
|
|
- forcetypeassert
|
|
## - funlen # checks function length. don't care
|
|
# - gci # not sure why this is complaining
|
|
## - gochecknoglobals # too common
|
|
- gochecknoinits
|
|
# - gocognit # too many findings, will take time to evaluate
|
|
- goconst
|
|
- gocritic
|
|
## - gocyclo # checks cyclomatic complexity. don't care
|
|
# - godot # too many false positives
|
|
# - godox # doesn't allow TODO comments. We allow those to be committed.
|
|
# - goerr113 # good practice, but it doesn't recognize that we're already wrapping errors with merry
|
|
## - gofmt # checks code is formatted, handled by make prep
|
|
- gofumpt
|
|
- goheader
|
|
## - goimports # checks import order. We're not using goimports
|
|
# - gomnd # too aggressive
|
|
- gomoddirectives
|
|
# - gomodguard
|
|
- goprintffuncname
|
|
- gosec
|
|
- grouper
|
|
- importas
|
|
# - ireturn # there are valid use cases for this pattern. too strict.
|
|
## - lll # checks line length. not enforced
|
|
# - maintidx # look at this later
|
|
- makezero
|
|
## - maligned # optimizies struct field order, but structs are usually ordered for legibility
|
|
- misspell
|
|
- nakedret
|
|
# - nestif # need to evaluate the findings
|
|
- nilerr
|
|
- nilnil
|
|
# - nlreturn # a little too aggressive. wsl covers the same ground.
|
|
- noctx
|
|
- nolintlint
|
|
# - paralleltest # look at this later
|
|
# - prealloc # slice optimizations, but promotes too much premature optimization
|
|
- predeclared
|
|
- promlinter
|
|
- revive
|
|
- rowserrcheck
|
|
- sqlclosecheck
|
|
- stylecheck
|
|
- tagliatelle
|
|
- thelper
|
|
- tparallel
|
|
- unconvert
|
|
- unparam
|
|
# - varnamelen # take a look later
|
|
- wastedassign
|
|
- whitespace
|
|
# - wrapcheck # way too aggressive
|
|
- wsl
|
|
## - unparam # too many false positives
|
|
## - whitespace # not enforced
|
|
disable-all: true
|
|
# presets:
|
|
# - bugs
|
|
# - unused
|
|
# fast: false
|
|
|
|
|
|
issues:
|
|
# List of regexps of issue texts to exclude, empty list by default.
|
|
# But independently from this option we use default exclude patterns,
|
|
# it can be disabled by `exclude-use-default: false`. To list all
|
|
# excluded by default patterns execute `golangci-lint run --help`
|
|
# exclude:
|
|
# - abcdef
|
|
|
|
# Excluding configuration per-path, per-linter, per-text and per-source
|
|
exclude-rules:
|
|
# Explicitly exclude the typecheck plugin. There is some bug in golangci which is
|
|
# enabling this checker, even though it isn't listed above.
|
|
# Exclude some linters from running on tests files.
|
|
- path: _test\.go
|
|
linters:
|
|
- gocyclo
|
|
- errcheck
|
|
- dupl
|
|
- gosec
|
|
- exportloopref
|
|
- gochecknoinits
|
|
- gochecknoglobals
|
|
- wsl
|
|
- nlreturn
|
|
- errchkjson
|
|
- forcetypeassert
|
|
- path: cmd
|
|
linters:
|
|
# init(), globals, and prints are pretty common in main packages
|
|
- gochecknoinits
|
|
- gochecknoglobals
|
|
- forbidigo
|
|
|
|
# Exclude known linters from partially hard-vendored code,
|
|
# which is impossible to exclude via "nolint" comments.
|
|
# - path: internal/hmac/
|
|
# text: "weak cryptographic primitive"
|
|
# linters:
|
|
# - gosec
|
|
|
|
# Exclude some staticcheck messages
|
|
# - linters:
|
|
# - staticcheck
|
|
# text: "SA9003:"
|
|
|
|
|
|
# Independently from option `exclude` we use default exclude patterns,
|
|
# it can be disabled by this option. To list all
|
|
# excluded by default patterns execute `golangci-lint run --help`.
|
|
# Default value for this option is true.
|
|
# exclude-use-default: false
|
|
|
|
# Maximum issues count per one linter. Set to 0 to disable. Default is 50.
|
|
# max-issues-per-linter: 0
|
|
|
|
# Maximum count of issues with the same text. Set to 0 to disable. Default is 3.
|
|
# max-same-issues: 0
|
|
|
|
# Show only new issues: if there are unstaged changes or untracked files,
|
|
# only those changes are analyzed, else only changes in HEAD~ are analyzed.
|
|
# It's a super-useful option for integration of golangci-lint into existing
|
|
# large codebase. It's not practical to fix all existing issues at the moment
|
|
# of integration: much better don't allow issues in new code.
|
|
# Default is false.
|
|
new: false
|
|
|
|
# Show only new issues created after git revision `REV`
|
|
# new-from-rev: REV
|
|
|
|
# Show only new issues created in git patch with set file path.
|
|
# new-from-patch: path/to/patch/file |