.golangci.yml

# options for analysis running
run:
  # default concurrency is a available CPU number
  concurrency: 4

  # timeout for analysis, e.g. 30s, 5m, default is 1m
  timeout: 10m
linters:
  # please, do not use `enable-all`: it's deprecated and will be removed soon.
  # inverted configuration with `enable-all` and `disable` is not scalable during updates of golangci-lint
  disable-all: true
  enable:
    - typecheck
    - dupl
    - goprintffuncname
    - govet
    - nolintlint
    #- rowserrcheck
    - gofmt
    - revive
    - goimports
    - misspell
    - bodyclose
    - unconvert
    - ineffassign
    - staticcheck
    - exportloopref
    - depguard
    - dogsled
    - errcheck
    #- funlen
    - gci
    - goconst
    - gocritic
    - gocyclo
    - gosimple
    - stylecheck
    - unused
    - unparam
    - unconvert
    - whitespace

issues:
  include:
  - EXC0002 # disable excluding of issues about comments from golint
  # Excluding configuration per-path, per-linter, per-text and per-source
  exclude-rules:
    - path: _test\.go
      linters:
        - gomnd
        - dupl
    # Exclude because: "string `Deployment` has 3 occurrences, but such constant `Deployment` already exists"
    # but we don't need to make them constants
    - path: controllers/keda/transform/transform.go
      linters:
        - goconst
    # Exclude because: "string `Deployment` has 3 occurrences, but such constant `Deployment` already exists"
    # but we don't need to make them constants
    # and:
    # "exported: type name will be used as keda.KedaControllerReconciler by other packages, and that stutters;
    # consider calling this ControllerReconciler (revive)"
    - path: controllers/keda/kedacontroller_controller.go
      linters:
        - goconst
        - revive
    # Exclude for controllers, reason:
    # "controllers/configmap_controller.go:49: 49-80 lines are duplicate of `controllers/secret_controller.go:49-80`"
    - path: _controller.go
      linters:
        - dupl
    # Exclude because: "controllers/suite_test.go:157:63: `getObject` - `namespace` always receives `namespace` (`"keda"`)"
    - path: controllers/keda/suite_test.go
      linters:
        - unparam
    # Exclude because: "declaration has 3 blank identifiers" -> it's ok, we don't need them
    - path: resources/resources_handler.go
      linters:
        - dogsled
    - path: main.go
      linters:
        - gci
    # https://github.com/go-critic/go-critic/issues/926
    - linters:
        - gocritic
      text: "unnecessaryDefer:"
    - linters:
       - stylecheck
      text: "ST1000:"
    # Exclude some linters from running on test files.
    - path: _test\.go$
      text: "dot-imports: should not use dot imports"

linters-settings:
  funlen:
    lines: 80
    statements: 40
  gci:
    sections:
      - standard
      - default
      - prefix(github.com/kedacore/keda)
  # copied from https://github.com/kedacore/keda/blob/1ab35e75a63b3b33ca59552dda93220fd916a05d/.golangci.yml#L163-L168
  depguard: #https://github.com/kedacore/keda/issues/4980
    rules:
      main:
        deny:
          - pkg: sync/atomic
            desc: "use type-safe atomics from go.uber.org/atomic"