.golangci.yml

#
# This file lives in the github.com/metal-toolbox/golangci-lint-config repo.
#
# Do not edit this file outside of this repo otherwise we will be grumpy.
# Seriously though, this is meant to help promote a "standard" config and coding style.
# If you don't like something, lets have a discussion in GitHub issues!
#

linters-settings:
  dupl:
    threshold: 125
  goconst:
    min-len: 2
    min-occurrences: 2
  gocritic:
    enabled-tags:
      - experimental
      - performance
      - style
    disabled-checks:
      - whyNoLint
      - wrapperFunc
  gocyclo:
    min-complexity: 15
  gofumpt:
    extra-rules: true
  govet:
    enable:
      - shadow
  lll:
    line-length: 140
  misspell:
    locale: US
  revive:
    confidence: 0

linters:
  enable-all: true
  disable-all: false
  # Linters we don't like
  # Comments help explain why its disabled or point at ones we should not disable but will take a little work
  # If its not commented its likely because its just too annoying or we don't find useful
  disable:
    - cyclop
    - depguard
    - errname # maybe should be enabled
    - execinquery
    - exhaustruct
    - exportloopref
    - forbidigo
    - funlen
    - gci
    - gochecknoglobals
    - gochecknoinits
    - gocognit
    - goconst
    - godot
    - godox
    - gomnd
    - inamedparam
    - interfacebloat
    - ireturn # should be enabled, ironlib needs some changes
    - intrange
    - lll # not previously enabled, ironlib and mctl both fail this
    - mnd
    - nestif
    - nilnil
    - nlreturn
    - nolintlint
    - nonamedreturns # FIXME: named returns are unmaintainable
    - paralleltest
    - perfsprint
    - tagliatelle
    - tenv # should be enabled
    - testpackage
    - testifylint # should be enabled
    - thelper # should be enabled
    - varnamelen
    - wrapcheck
    - wsl

issues:
  exclude-files:
    - ".*_test\\.go"
  exclude-dirs:
    - fixtures
  exclude-rules:
    - linters:
        - stylecheck
      text: "ST1016"