dive/.golangci.yaml
Alex Goodman d5e8a92968
Rework CI validation workflow and makefile (#460)
* rework CI validation workflow and makefile

* enable push

* fix job names

* fix license check

* fix snapshot builds

* fix acceptance tests

* fix linting

* disable pull request event

* rework windows runner caching

* disable release pipeline and add issue templates
2023-07-06 22:01:46 -04:00

74 lines
2.5 KiB
YAML

# TODO: enable this when we have coverage on docstring comments
#issues:
# # The list of ids of default excludes to include or disable.
# include:
# - EXC0002 # disable excluding of issues about comments from golint
linters-settings:
funlen:
# Checks the number of lines in a function.
# If lower than 0, disable the check.
# Default: 60
# TODO: drop this down over time...
lines: 110
# Checks the number of statements in a function.
# If lower than 0, disable the check.
# Default: 40
statements: 60
# TODO: use the default linters for now, but include these over time
#linters:
# # inverted configuration with `enable-all` and `disable` is not scalable during updates of golangci-lint
# disable-all: true
# enable:
# - asciicheck
# - bodyclose
# - depguard
# - dogsled
# - dupl
# - errcheck
# - exportloopref
# - funlen
# - gocognit
# - goconst
# - gocritic
# - gocyclo
# - gofmt
# - goimports
# - goprintffuncname
# - gosec
# - gosimple
# - govet
# - ineffassign
# - misspell
# - nakedret
# - nolintlint
# - revive
# - staticcheck
# - stylecheck
# - typecheck
# - unconvert
# - unparam
# - unused
# - whitespace
# do not enable...
# - gochecknoglobals
# - gochecknoinits # this is too aggressive
# - godot
# - godox
# - goerr113
# - golint # deprecated
# - gomnd # this is too aggressive
# - interfacer # this is a good idea, but is no longer supported and is prone to false positives
# - lll # without a way to specify per-line exception cases, this is not usable
# - maligned # this is an excellent linter, but tricky to optimize and we are not sensitive to memory layout optimizations
# - nestif
# - prealloc # following this rule isn't consistently a good idea, as it sometimes forces unnecessary allocations that result in less idiomatic code
# - scopelint # deprecated
# - testpackage
# - wsl # this doens't have an auto-fixer yet and is pretty noisy (https://github.com/bombsimon/wsl/issues/90)
# - varcheck # deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
# - deadcode # deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
# - structcheck # deprecated (since v1.49.0) due to: The owner seems to have abandoned the linter. Replaced by unused.
# - rowserrcheck # we're not using sql.Rows at all in the codebase