osv-scalibr/.golangci.yaml at main · i-bs/osv-scalibr · GitHub

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
# TODO(#274): this should match the configuration of osv-scanner for the most part
#   currently it does not because using that config raises type errors,
#   in addition to some of the rules having violations that need addressing,
#   so the current focus is on getting as many linters as possible running
version: "2"

linters:
  default: all
  disable:
    - cyclop           #
    - depguard         #
    - dupl             # Some interfaces end up having several lines of similar code as others
    - err113           #
    - exhaustruct      #
    - exhaustive       # TODO(#1225): work on enabling this
    - forcetypeassert  #
    - funcorder        #
    - funlen           #
    - gochecknoglobals #
    - gocognit         #
    - goconst          #
    - gocyclo          #
    - godot            #
    - godox            #
    - gosec            # TODO(#274): work on enabling this
    - ireturn          #
    - lll              #
    - maintidx         #
    - mnd              # Not every number is magic
    - nestif           #
    - nilnesserr       # TODO(#274): work on enabling this
    - nilnil           # We consider this a valid pattern to use sometimes
    - nlreturn         # Not feasible until it's supported by the internal linter
    - noinlineerr      #
    - nonamedreturns   #
    - paralleltest     # Parallel tests mixes up log lines of multiple tests in the internal test runner
    - prealloc         # We don't want to preallocate all the time
    - protogetter      # TODO(#274): work on enabling this
    - recvcheck        # TODO(#274): work on enabling this
    - tagliatelle      #
    - testpackage      #
    - tparallel        # Parallel tests mixes up log lines of multiple tests in the internal test runner
    - varnamelen       #
    - wrapcheck        #
    - wsl              #
    - wsl_v5           #
  settings:
    forbidigo:
      forbid:
        # Parallel tests mixes up log lines of multiple tests in the internal test runner
        - pattern: ^testing.T.Parallel$
          pkg: ^testing$
      analyze-types: true
    gocritic:
      disabled-checks:
        - ifElseChain
    revive:
      rules:
        - 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: import-alias-naming
          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: unreachable-code
          disabled: false
        - name: use-any
          disabled: false
        - name: var-declaration
          disabled: false
        - name: var-naming
          disabled: false
          arguments:
            - [] # AllowList
            - [] # DenyList
            - - skip-package-name-checks: true
    staticcheck:
      checks:
        - all
        - -SA5011 # seems prone to false positives in tests
        - -QF1001 # apply De Morgan's law
        - -QF1003 # use tagged switch on prefix
  exclusions:
    generated: lax
    presets:
      - common-false-positives
      - legacy
      - std-error-handling
    rules:
      - path: veles/
        linters:
          - forbidigo  # Parallel tests are fine and in some places needed for Veles.
      - path: velestest/
        linters:
          - forbidigo  # Parallel tests are fine and in some places needed for Veles.
    paths:
      - third_party$
      - builtin$
      - examples$

formatters:
  enable:
    - gofmt
    - goimports
  settings:
    gofmt:
      simplify: false
  exclusions:
    generated: lax
    paths:
      - third_party$
      - builtin$
      - examples$

issues:
  max-issues-per-linter: 0
  max-same-issues: 0