.golangci.yml

---
linters-settings:
  exhaustive:
    default-signifies-exhaustive: true
  cyclop:
    max-complexity: 30


  depguard:
    rules:
      main:
        deny:
          - pkg: "github.com/pkg/errors"
            desc: "use 'errors' instead"
          - pkg: "github.com/gogo/protobuf/proto"
            desc: "use 'github.com/golang/protobuf/proto' instead"
          - pkg: "github.com/percona/go-mysql/log/slow"
            desc: "use only forked parser"
          - pkg: "gopkg.in/yaml.v2"
            desc: "use 'gopkg.in/yaml.v3' instead"

  gci:
    sections:
      - standard # Captures all standard packages if they do not match another section.
      - default # Contains all imports that could not be matched to another section type.
      - prefix(github.com/percona/everest-operator) # Groups all local imports.

  godot:
    capital: true

  goimports:
    local-prefixes: github.com/percona/everest-operator

  ireturn:
    allow:
      - anon
      - error
      - empty
      - stdlib
      - github.com/charmbracelet/bubbletea.Model

  lll:
    line-length: 170
    tab-width: 4

  tagliatelle:
    # Check the struck tag name case.
    case:
      rules:
        avro: snake
        bson: camel
        # Disabled because we have too many errors at the moment
        json: ""
        # Disabled because we have too many errors at the moment
        yaml: ""
        xml: camel

  unused:
    check-exported: false #TODO: set true for agent

  unparam:
    check-exported: true

linters:
  enable-all: true
  disable:
    # keep sorted
    - exhaustivestruct  # too annoying
    - funlen            # useless
    - gochecknoglobals  # mostly useless
    - goerr113          # useless
    - gochecknoinits    # we use init functions
    - gomnd             # we are using numbers in many cases
    - gomoddirectives   # we use replace directives
    - ifshort           # a lot of false positives
    - interfacer        # deprecated
    - maligned          # deprecated
    - nlreturn          # too annoying
    - scopelint         # too many false positives
    - varnamelen        # useless
    - wrapcheck         # we do not use wrapping everywhere
    - wsl               # too annoying
    - exhaustruct       # too many files to fix/nolint
    - goerr113          # extra work & poor benefit
    - varcheck          # deprecated
    - structcheck       # deprecated
    - golint
    - nosnakecase
    - deadcode
    - exhaustive
    - godox

run:
  timeout: 5m
  skip-dirs:
    - agent/agents/mongodb/internal/
    - qan-api2/migrations/

issues:
  exclude-use-default: false
  exclude:
    # gas: Duplicated errcheck checks
    - 'G104: Errors unhandled'
    # golint: Methods for implementing interfaces are not documented
    - 'exported method `.+` should have comment or be unexported'
    - 'comment on exported method `.+` should be of the form `.+ ...`'
    # golint: We have to return unexported types due to oneof implementation
    - 'exported method .+ returns unexported type .+, which can be annoying to use'
  exclude-rules:
    - path: _test\.go
      linters:
        # keep sorted
        - exhaustivestruct # very annoying
        - funlen           # tests may be long
        - gocognit         # triggered by subtests
        - gomnd            # tests are full of magic numbers
        - testpackage      # senseless
        - unused           # very annoying false positive: https://github.com/golangci/golangci-lint/issues/791
        - lll              # tests often require long lines