.golangci.yaml

output:
  sort-results: true
linters:
  enable:
    - gofmt
    - goimports
  # prettier-ignore
  disable:
    - tagliatelle      # we're parsing data from external sources (g-rath)
    - varnamelen       # maybe later (g-rath)
    - exhaustruct      # overkill (g-rath)
    - forcetypeassert  # too hard (g-rath)
    - lll              # line length is hard (g-rath)
    - godox            # to-do comments are fine (g-rath)
    - godot            # comments are fine without full stops (g-rath)
    - mnd              # not every number is magic (g-rath)
    - wsl              # disagree with, for now (g-rath)
    - ireturn          # disagree with, sort of (g-rath)
    - gochecknoglobals # disagree with, for non changing variables (another-rex)
    - wrapcheck        # too difficult, will re-add later (another-rex)
    - testpackage      # will re-add later (another-rex)
    - err113           # will re-add later (another-rex)
    - nonamedreturns   # disagree with, for now (another-rex)
    - goconst          # not everything should be a constant
  presets:
    - bugs
    - comment
    - error
    - performance
    - sql
    - style
    - test
    - unused

linters-settings:
  forbidigo:
    forbid:
      - p: ^testing.T.Skip
        pkg: ^testing$
        msg: "go-snaps needs to know the test has been skipped, so use `testutility.Skip` instead"
    analyze-types: true
  govet:
    settings:
      printf:
        funcs:
          - (github.com/google/osv-scanner/v2/pkg/reporter.Reporter).Errorf
          - (github.com/google/osv-scanner/v2/pkg/reporter.Reporter).Warnf
          - (github.com/google/osv-scanner/v2/pkg/reporter.Reporter).Verbosef
          - (github.com/google/osv-scanner/v2/pkg/reporter.Reporter).Infof
  depguard:
    rules:
      regexp:
        files:
          - "!**/internal/cachedregexp/**"
          - "!**/main_test.go"
        deny:
          - pkg: "regexp"
            desc: "Use github.com/google/osv-scanner/v2/internal/cachedregexp instead"
  gocritic:
    disabled-checks:
      - ifElseChain
  revive:
    # enable-all-rules: true
    rules:
      # Overrides ----
      # This is just here for documentation purposes, as all rules are disabled by default
      - name: increment-decrement
        disabled: true
      # Defaults ----
      - name: blank-imports
        disabled: false
      - name: context-as-argument
        disabled: false
      - name: context-keys-type
        disabled: false
      - name: dot-imports
        disabled: false
      - name: empty-block
        disabled: false
      - name: error-naming
        disabled: false
      - name: error-return
        disabled: false
      - name: error-strings
        disabled: false
      - name: errorf
        disabled: false
      - name: exported
        disabled: false
      - name: indent-error-flow
        disabled: false
      - name: package-comments
        disabled: false
      - name: range
        disabled: false
      - name: receiver-naming
        disabled: false
      - name: redefines-builtin-id
        disabled: false
      - name: superfluous-else
        disabled: false
      - name: time-naming
        disabled: false
      - name: unexported-return
        disabled: false
      - name: unreachable-code
        disabled: false
      - name: unused-parameter
        disabled: false
      - name: var-declaration
        disabled: false
      - name: var-naming
        disabled: false
  nlreturn:
    # Size of the block (including return statement that is still "OK")
    # so no return split required.
    # Default: 1
    block-size: 2

issues:
  exclude-rules:
    - path: pkg/reporter
      linters:
        - dupl
    - path: _test\.go
      linters:
        - dupl
  exclude-dirs:
    - internal/thirdparty/xml