.golangci.yaml

run:
  timeout: 5m

issues:
  max-same-issues: 0 # don't hide issues in CI runs because they are the same type
  exclude-dirs:
    - internal/gojsonschema
    - internal/gqlparser
    - internal/jwx
  exclude-rules:
    - path: ast/
      linters:
        - staticcheck
      text: "SA1019"
    - path: bundle/
      linters:
        - staticcheck
      text: "SA1019"
    - path: capabilities/
      linters:
        - staticcheck
      text: "SA1019"
    - path: compile/
      linters:
        - staticcheck
      text: "SA1019"
    - path: config/
      linters:
        - staticcheck
      text: "SA1019"
    - path: cover/
      linters:
        - staticcheck
      text: "SA1019"
    - path: debug/
      linters:
        - staticcheck
      text: "SA1019"
    - path: dependencies/
      linters:
        - staticcheck
      text: "SA1019"
    - path: download/
      linters:
        - staticcheck
      text: "SA1019"
    - path: format/
      linters:
        - staticcheck
      text: "SA1019"
    - path: hooks/
      linters:
        - staticcheck
      text: "SA1019"
    - path: ir/
      linters:
        - staticcheck
      text: "SA1019"
    - path: keys/
      linters:
        - staticcheck
      text: "SA1019"
    - path: loader/
      linters:
        - staticcheck
      text: "SA1019"
    - path: logging/
      linters:
        - staticcheck
      text: "SA1019"
    - path: metrics/
      linters:
        - staticcheck
      text: "SA1019"
    - path: plugins/
      linters:
        - staticcheck
      text: "SA1019"
    - path: profiler/
      linters:
        - staticcheck
      text: "SA1019"
    - path: refactor/
      linters:
        - staticcheck
      text: "SA1019"
    - path: repl/
      linters:
        - staticcheck
      text: "SA1019"
    - path: rego/
      linters:
        - staticcheck
      text: "SA1019"
    - path: resolver/
      linters:
        - staticcheck
      text: "SA1019"
    - path: runtime/
      linters:
        - staticcheck
      text: "SA1019"
    - path: schemas/
      linters:
        - staticcheck
      text: "SA1019"
    - path: sdk/
      linters:
        - staticcheck
      text: "SA1019"
    - path: server/
      linters:
        - staticcheck
      text: "SA1019"
    - path: storage/
      linters:
        - staticcheck
      text: "SA1019"
    - path: tester/
      linters:
        - staticcheck
      text: "SA1019"
    - path: topdown/
      linters:
        - staticcheck
      text: "SA1019"
    - path: tracing/
      linters:
        - staticcheck
      text: "SA1019"
    - path: types/
      linters:
        - staticcheck
      text: "SA1019"
    - path: util/
      linters:
        - staticcheck
      text: "SA1019"
    - path: version/
      linters:
        - staticcheck
      text: "SA1019"

linters-settings:
  lll:
    line-length: 200
  gocritic:
    disabled-checks:
      - appendAssign
      # NOTE(ae): this one should be enabled, but there were too
      # many violations to fix in one go... revisit later
      - singleCaseSwitch
      # Reasonable rule, but not sure what to replace with in
      # many locations, so disabling for now
      - exitAfterDefer
      # The following 3 rules are disabled from the perfomance tag
      # enabled further down. The first two are reasonable, but not
      # super important. appendCombine is really nice though! And
      # should be enabled. Just many places to fix..
      - hugeParam
      - preferFprint
      - appendCombine
    enabled-checks:
      # NOTE that these are rules enabled in addition to the default set
      - filepathJoin
      - dupImport
      - redundantSprint
      - stringConcatSimplify
    enabled-tags:
      - performance
    settings:
      ifElseChain:
        # ridiculous value set for now, but this should be
        # lowered to something more reasonable, as the rule
        # is reasonable (replace long if-else chains with
        # switch)... just too many violations right now
        minThreshold: 10
  govet:
    enable:
      - deepequalerrors
      - nilness
  perfsprint:
    # only rule disabled by default, but it's a good one
    err-error: true
  revive:
    rules:
    # this mainly complains about us using min/max for variable names,
    # which seems like an unlikely source of actual issues
     - name: redefines-builtin-id
       disabled: true

linters:
  disable-all: true
  enable:
    - mirror
    - errcheck
    - govet
    - ineffassign
    - intrange
    - revive # replacement for golint
    - gofmt
    - goimports
    - unused
    - misspell
    - usetesting
    - typecheck
    - staticcheck
    - gosimple
    - prealloc
    - unconvert
    - copyloopvar
    - perfsprint
    - gocritic
    # - gosec # too many false positives