.golangci.yml

run:
  # timeout for analysis
  deadline: 10m

linters-settings:
  govet:
    # Don't report about shadowed variables
    check-shadowing: false
  gofmt:
    # simplify code: gofmt with `-s` option, true by default
    simplify: true
  whitespace:
    multi-func: true
    multi-if: true

linters:
  enable-all: true
  disable:
    # Global variables are used in many places throughout the code base.
    - gochecknoglobals

    # Some lines are over 80 characters on purpose and we don't want to make them
    # even longer by marking them as 'nolint'.
    - lll

    # We want to allow short variable names.
    - varnamelen

    # We want to allow TODOs.
    - godox

    # We have long functions, especially in tests. Moving or renaming those would
    # trigger funlen problems that we may not want to solve at that time.
    - funlen

    # Disable for now as we haven't yet tuned the sensitivity to our codebase
    # yet. Enabling by default for example, would also force new contributors to
    # potentially extensively refactor code, when they want to smaller change to
    # land.
    - gocyclo
    - gocognit
    - cyclop

    # Instances of table driven tests that don't pre-allocate shouldn't trigger
    # the linter.
    - prealloc

    # Init functions are used by loggers throughout the codebase.
    - gochecknoinits

    # Causes stack overflow, see https://github.com/polyfloyd/go-errorlint/issues/19.
    - errorlint

    # Deprecated linters. See https://golangci-lint.run/usage/linters/.
    - interfacer
    - golint
    - maligned
    - scopelint

    # New linters that need a code adjustment first.
    - wrapcheck
    - nolintlint
    - paralleltest
    - tparallel
    - testpackage
    - gofumpt
    - gomoddirectives
    - ireturn
    - maintidx
    - nlreturn
    - dogsled
    - gci
    - containedctx
    - contextcheck
    - errname
    - exhaustivestruct
    - goerr113
    - gomnd
    - ifshort
    - noctx
    - nestif
    - wsl
    - exhaustive
    - forcetypeassert
    - nilerr
    - nilnil
    - stylecheck
    - thelper
    - exhaustruct
    - nosnakecase

issues:
  exclude-rules:
    # Exclude gosec from running for tests so that tests with weak randomness
    # (math/rand) will pass the linter.
    - path: _test\.go
      linters:
        - gosec
        - errcheck
        - dupl

        # Instances of table driven tests that don't pre-allocate shouldn't
        # trigger the linter.
        - prealloc