705c72d293
Running with default set of checks, except disabling the following
- appendAssign: append result not assigned to the same slice (diagnostics check, many false positives)
- captLocal: using capitalized names for local variables (style check, too opinionated)
- commentFormatting: not having a space between `//` and comment text (style check, too opinionated)
- exitAfterDefer: log.Fatalln will exit, and `defer func(){...}(...)` will not run (diagnostics check, to be revisited)
- ifElseChain: rewrite if-else to switch statement (style check, many occurrences and a bit opinionated, to be revisited)
- singleCaseSwitch: should rewrite switch statement to if statement (style check, many occurrences and a bit opinionated, to be revisited)
109 lines
2.6 KiB
YAML
109 lines
2.6 KiB
YAML
# golangci-lint configuration options
|
|
|
|
linters:
|
|
enable:
|
|
- errcheck
|
|
- goimports
|
|
- revive
|
|
- ineffassign
|
|
- govet
|
|
- unconvert
|
|
- staticcheck
|
|
- gosimple
|
|
- stylecheck
|
|
- unused
|
|
- misspell
|
|
- gocritic
|
|
#- prealloc
|
|
#- maligned
|
|
disable-all: true
|
|
|
|
issues:
|
|
# Enable some lints excluded by default
|
|
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
|
|
|
|
exclude-rules:
|
|
|
|
- linters:
|
|
- staticcheck
|
|
text: 'SA1019: "github.com/rclone/rclone/cmd/serve/httplib" is deprecated'
|
|
|
|
# don't disable the revive messages about comments on exported functions
|
|
include:
|
|
- EXC0012
|
|
- EXC0013
|
|
- EXC0014
|
|
- EXC0015
|
|
|
|
run:
|
|
# timeout for analysis, e.g. 30s, 5m, default is 1m
|
|
timeout: 10m
|
|
|
|
linters-settings:
|
|
revive:
|
|
# setting rules seems to disable all the rules, so re-enable them here
|
|
rules:
|
|
- name: blank-imports
|
|
disabled: false
|
|
- name: context-as-argument
|
|
disabled: false
|
|
- name: context-keys-type
|
|
disabled: false
|
|
- name: dot-imports
|
|
disabled: false
|
|
- name: empty-block
|
|
disabled: true
|
|
- name: error-naming
|
|
disabled: false
|
|
- name: error-return
|
|
disabled: false
|
|
- name: error-strings
|
|
disabled: false
|
|
- name: errorf
|
|
disabled: false
|
|
- name: exported
|
|
disabled: false
|
|
- name: increment-decrement
|
|
disabled: true
|
|
- name: indent-error-flow
|
|
disabled: false
|
|
- name: package-comments
|
|
disabled: false
|
|
- name: range
|
|
disabled: false
|
|
- name: receiver-naming
|
|
disabled: false
|
|
- name: redefines-builtin-id
|
|
disabled: true
|
|
- name: superfluous-else
|
|
disabled: true
|
|
- name: time-naming
|
|
disabled: false
|
|
- name: unexported-return
|
|
disabled: false
|
|
- name: unreachable-code
|
|
disabled: true
|
|
- name: unused-parameter
|
|
disabled: true
|
|
- name: var-declaration
|
|
disabled: false
|
|
- name: var-naming
|
|
disabled: false
|
|
stylecheck:
|
|
# Only enable the checks performed by the staticcheck stand-alone tool,
|
|
# as documented here: https://staticcheck.io/docs/configuration/options/#checks
|
|
checks: ["all", "-ST1000", "-ST1003", "-ST1016", "-ST1020", "-ST1021", "-ST1022", "-ST1023"]
|
|
gocritic:
|
|
disabled-checks:
|
|
- appendAssign
|
|
- captLocal
|
|
- commentFormatting
|
|
- exitAfterDefer
|
|
- ifElseChain
|
|
- singleCaseSwitch
|