.golangci.yml

# Options for analysis running.
run:
  go: "1.22"
  skip-files:
    - "zz_generated.*\\.go$"
    - "conversion\\.go"
    - "vendored_openapi\\.go$"
    - ".*_test\\.go$"
  allow-parallel-runners: true
issues:
  include:
    - EXC0012
    - EXC0014
  exclude-rules:
    - path: _test\.go
      linters:
        - testpackage
        - gochecknoglobals
    - path: controlplane/api/v1alpha1/*
      linters:
        - gochecknoglobals
    - source: "^//\\+kubebuilder:"
      linters:
        - lll
    - path: _webhook\.go
      linters:
        - dupl
        - wrapcheck
    - text: var-naming
      linters:
        - revive
    - text: ST1003
      linters: 
        - stylecheck
    - text: G114    # Needs further discussion to confirm disabling 
      linters:
        - gosec  
    - text: G402    # Needs futher discussion
      path: api/v1alpha1
      linters:
        - gosec
    - text: unexported-return
      path: pkg/rke2
      linters:
        - revive
linters-settings:
  funlen:
    lines: 110
    statements: 60
  lll:
    line-length: 150
  gomnd:
    ignored-numbers:
      - "5"
      - "4"
      - "3"
  goimports:
    local-prefixes: github.com/rancher/cluster-api-provider-rke2
  gci:
    custom-order: true
    sections:
      - "standard"
      - "blank"
      - "dot"
      - "default"
      - "prefix(sigs.k8s.io/cluster-api)"
      - "prefix(github.com/rancher/cluster-api-provider-rke2)"
    wsl:
      force-err-cuddling: false
  revive:
      rules:
        # The following rules are recommended https://github.com/mgechev/revive#recommended-configuration
        - name: blank-imports
        - name: context-as-argument
        - name: context-keys-type
        - name: dot-imports
        - name: error-return
        - name: error-strings
        - name: error-naming
        - name: exported
        - name: if-return
        - name: increment-decrement
        - name: var-naming
        - name: var-declaration
        - name: package-comments
        - name: range
        - name: receiver-naming
        - name: time-naming
        - name: unexported-return
        - name: indent-error-flow
        - name: errorf
        - name: empty-block
        - name: superfluous-else
        - name: unused-parameter
        - name: unreachable-code
        - name: redefines-builtin-id
        #
        # Rules in addition to the recommended configuration above.
        #
        - name: bool-literal-in-expr
        - name: constant-logical-expr
linters:
  enable-all: true
  disable:
    - nolintlint
    - tagliatelle
    - exhaustruct
    - exhaustive
    - exhaustivestruct
    - maligned
    - funlen
    - wrapcheck
    - goerr113  # this one should probably be re-enabled soon
    - paralleltest
    - varnamelen
    - cyclop
    - gochecknoglobals
    - gochecknoinits
    - nonamedreturns
    - gocritic
    - depguard
    - ireturn     # Needs further discussion
    - maintidx    # Needs further discussion
    - nestif      # Needs further discussion
    - gocognit    # Needs further discussion
    - containedctx # Needs further discussion
    # The following are all deprecated linters
    - deadcode
    - ifshort
    - varcheck
    - scopelint
    - interfacer
    - structcheck
    - golint
    - nosnakecase
    # Disabled because of generics
    - rowserrcheck
    - sqlclosecheck
    - wastedassign