.golangci.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