.golangci.yaml

issues:
  exclude-dirs:
    - internal/kubernetes_vendor
  exclude-files:
    - "pkg/diff/internal/fieldmanager/borrowed_.*\\.go$"
  max-issues-per-linter: 0
  max-same-issues: 0
linters:
  enable:
    - errcheck
    - errorlint
    - gocritic
    - gofumpt
    - goimports
    - gomodguard
    - gosimple
    - govet
    - importas
    - ineffassign
    - misspell
    # Disabled because of https://github.com/argoproj/argo-cd/issues/21705
    # - nolintlint
    - perfsprint
    - revive
    - staticcheck
    - testifylint
    - thelper
    - unparam
    - unused
    - usestdlibvars
    - whitespace
linters-settings:
  gocritic:
    disabled-checks:
      - appendAssign
      - assignOp # Keep it disabled for readability
      - exitAfterDefer
      - typeSwitchVar
  goimports:
    local-prefixes: github.com/argoproj/gitops-engine
  importas:
    alias:
      - alias: appsv1
        pkg: k8s.io/api/apps/v1
      - alias: corev1
        pkg: k8s.io/api/core/v1
      - alias: apierrors
        pkg: k8s.io/apimachinery/pkg/api/errors
      - alias: apiextensionsv1
        pkg: k8s.io/apiextensions-apiserver/pkg/apis/apiextensions/v1
      - alias: metav1
        pkg: k8s.io/apimachinery/pkg/apis/meta/v1
      - alias: testingutils
        pkg: github.com/argoproj/gitops-engine/pkg/utils/testing
  perfsprint:
    # Optimizes even if it requires an int or uint type cast.
    int-conversion: true
    # Optimizes into `err.Error()` even if it is only equivalent for non-nil errors.
    err-error: true
    # Optimizes `fmt.Errorf`.
    errorf: true
    # Optimizes `fmt.Sprintf` with only one argument.
    sprintf1: true
    # Optimizes into strings concatenation.
    strconcat: true
  revive:
    # https://github.com/mgechev/revive/blob/master/RULES_DESCRIPTIONS.md
    rules:
      - name: bool-literal-in-expr
      - name: blank-imports
        disabled: true
      - name: context-as-argument
        arguments:
          - allowTypesBefore: '*testing.T,testing.TB'
      - name: context-keys-type
        disabled: true
      - name: dot-imports
      - name: duplicated-imports
      - name: early-return
        arguments:
          - 'preserveScope'
      - name: empty-block
        disabled: true
      - name: error-naming
        disabled: true
      - name: error-return
      - name: error-strings
        disabled: true
      - name: errorf
      - name: identical-branches
      - name: if-return
      - name: increment-decrement
      - name: indent-error-flow
        arguments:
          - 'preserveScope'
      - name: modifies-parameter
      - name: optimize-operands-order
      - name: range
      - name: receiver-naming
      - name: redefines-builtin-id
        disabled: true
      - name: redundant-import-alias
      - name: superfluous-else
        arguments:
          - 'preserveScope'
      - name: time-equal
      - name: time-naming
        disabled: true
      - name: unexported-return
        disabled: true
      - name: unnecessary-stmt
      - name: unreachable-code
      - name: unused-parameter
      - name: use-any
      - name: useless-break
      - name: var-declaration
      - name: var-naming
        disabled: true
  testifylint:
    enable-all: true
    disable:
      - go-require
run:
  timeout: 5m