Zabbix

# everything (linter definitions) should be sorted in alphabetical order by name

​

run:

  timeout: 1m

  allow-parallel-runners: true

​

output:

  sort-results: true

​

linters-settings:

        arguments:

          - 4

​

      - name: max-public-structs

        arguments:

          - 5

​

      # disabled

​

      - name: add-constant

        disabled: true

      - name: banned-characters # useless, as every char needs to be configured

        disabled: true

      # common to have a public faceing signature and internal uglier signature

      # Walk and walk (for example)

      - name: confusing-naming

        disabled: true

      - name: confusing-results

        disabled: true

      # cyclomatic complexity checks are enough, no need to complicate life

      # with two different metrics for complexity.

      - name: cognitive-complexity

        disabled: true

      # duplicate

      - name: cyclomatic

        disabled: true

      # duplicate

      - name: file-header

        disabled: true

      - name: flag-parameter

        disabled: true

      # cyclomatic complexety checks are enough

      - name: function-length

        disabled: true

      # duplicate

      - name: line-length-limit

        disabled: true

      # duplicate

      - name: unhandled-error

        disabled: true

      - name: package-comments

        disabled: true

​

  stylecheck:

    checks:

      - all

      - -ST1000 # Incorrect or missing package comment

      - -ST1003 # Poorly chosen identifier

    dot-import-whitelist: []

    http-status-code-whitelist: []

      # defaults

      - .Errorf(

      - errors.New(

      - errors.Unwrap(

      - .Wrap(

      - .Wrapf(

      - .WithMessage(

      - .WithMessagef(

      - .WithStack(

      - errs.New(

​

linters:

  enable-all: true

  disable:

    # gomodguard is better

    - depguard

​

    # always check marshal errors

    - errchkjson

​

    # triggers on switches with default case.

    - exhaustive

​

    # allow use go default zero values .

    - exhaustruct

​

    # cyclomatic complexity checkers already enforce limit on function length.

    # no need to count lines.

    - funlen

​

    - gofumpt

​

    # conflicts with gofumpt and gci. the tool still should be used but gci

    # needs be the last formatter.

    - goimports

​

    # not many cases when this is helpful were observed

​

    # revive warns about this

    - nakedret

​

    - testpackage

​

    # having one length vars is fine.

    - varnamelen

​

    # deprecated

​

issues:

  exclude-use-default: false

  max-issues-per-linter: 0 # unlimited

  max-same-issues: 0 # unlimited

  exclude-rules:

    - path: _test\.go

      linters:

        - errcheck

        - gocritic

        - revive

        - wrapcheck

​

severity:

  default-severity: error