.golangci.yml

linters-settings:
  dupl:
    threshold: 150
  funlen:
    lines: 100
    statements: 50
  goconst:
    min-len: 2
    min-occurrences: 2
  gocritic:
    enabled-tags:
      - diagnostic
      - experimental
      - opinionated
      - performance
      - style
    disabled-checks:
      - dupImport # https://github.com/go-critic/go-critic/issues/845
      - ifElseChain
      - octalLiteral
      - whyNoLint
      - wrapperFunc
      - unnamedResult
  gocognit:
    min-complexity: 30
  goimports:
    local-prefixes: github.com/Mellanox/network-operator
  golint:
    min-confidence: 0
  gomnd:
    settings:
      mnd:
        # don't include the "operation" and "assign"
        checks: argument,case,condition,return
  govet:
    check-shadowing: true
    settings:
      printf:
        funcs:
          - (github.com/rs/zerolog/zerolog.Event).Msgf
  gosec:
    excludes:
      - G115
  lll:
    line-length: 120
  misspell:
    locale: US
  prealloc:
    # Report preallocation suggestions only on simple loops that have no returns/breaks/continues/gotos in them.
    # True by default.
    simple: true
    range-loops: true # Report preallocation suggestions on range loops, true by default
    for-loops: false # Report preallocation suggestions on for loops, false by default

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:
    - bodyclose
    - dogsled
    - dupl
    - errcheck
    - exportloopref
    - funlen
      #- gochecknoinits
      #- goconst
    - gocritic
    - gocognit
    - gofmt
    - goimports
    - gomnd
    - goprintffuncname
    - gosec
    - gosimple
      #- govet
    - ineffassign
    - lll
    - misspell
    - nakedret
    - prealloc
    - revive
    - staticcheck
    - stylecheck
    - typecheck
    - unconvert
    - unparam
    - unused
    - whitespace

issues:
  # Disable default golangci exclusions.
  exclude-use-default: false
  # Don't deduplicate output.
  max-same-issues: 0
  max-issues-per-linter: 0

  # Excluding configuration per-path, per-linter, per-text and per-source
  exclude-rules:
    - path: _test\.go
      linters:
      - revive
      - stylecheck
      text: "should not use dot imports"
    - path: _test\.go
      linters:
        - gomnd
        - goconst
    - text: "Magic number: 1"
      linters:
        - gomnd
    # controller-gen generates zz_generated.deepcopy.go that doesn't comply with some golangci-lint checks
    - path: api/v1alpha1/zz_generated.deepcopy.go
      linters:
        - stylecheck
        - goimports