oxy/.golangci.yml at master · vulcand/oxy · 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
version: "2"

formatters:
  enable:
    - gci
    - gofumpt
  settings:
    gofumpt:
      extra-rules: false

linters:
  default: all
  disable:
    - bodyclose # Too many false positives: https://github.com/timakin/bodyclose/issues/30
    - cyclop # duplicate of gocyclo
    - dupl
    - err113
    - exhaustive
    - exhaustruct
    - forcetypeassert
    - gochecknoglobals # TODO(ldez) should be use on the project
    - ireturn
    - lll
    - mnd
    - musttag
    - nestif # TODO(ldez) should be use on the project
    - nilnil
    - nlreturn
    - noctx
    - nonamedreturns
    - paralleltest
    - prealloc
    - rowserrcheck # not relevant (SQL)
    - sqlclosecheck # not relevant (SQL)
    - tagliatelle
    - testpackage
    - tparallel
    - varnamelen
    - wrapcheck
    - noinlineerr
    - wsl # Deprecated
    - gomodguard # Deprecated

  settings:
    depguard:
      rules:
        main:
          deny:
            - pkg: github.com/instana/testify
              desc: not allowed
            - pkg: github.com/pkg/errors
              desc: Should be replaced by standard lib errors package
    funlen:
      lines: -1
      statements: 50
    goconst:
      min-len: 5
      min-occurrences: 3
    gocritic:
      disabled-checks:
        - sloppyReassign
        - rangeValCopy
        - octalLiteral
        - paramTypeCombine # already handle by gofumpt.extra-rules
        - httpNoBody
        - unnamedResult
        - deferInLoop # TODO(ldez) should be use on the project
      enabled-tags:
        - diagnostic
        - style
        - performance
      settings:
        hugeParam:
          sizeThreshold: 100
    gocyclo:
      min-complexity: 15
    godox:
      keywords:
        - FIXME
    gosec:
      excludes:
        - G705
    govet:
      disable:
        - fieldalignment
        - shadow
      enable-all: true
    misspell:
      locale: US
    perfsprint:
      err-error: true
      errorf: true
      sprintf1: true
      strconcat: false
    testifylint:
      disable:
        - go-require

  exclusions:
    warn-unused: true
    presets:
      - comments
      - std-error-handling
    rules:
      - linters:
          - canonicalheader
          - funlen
          - goconst
          - gosec
        path: .*_test.go
      - linters:
          - gosec
        path: testutils/.+
      - path: cbreaker/cbreaker_test.go # TODO(ldez) must be fixed
        text: 'statsNetErrors - threshold always receives 0.6'
      - path: buffer/buffer.go # TODO(ldez) must be fixed
        text: (cognitive|cyclomatic) complexity \d+ of func `\(\*Buffer\)\.ServeHTTP` is high
      - path: buffer/buffer.go # TODO(ldez) must be fixed
        text: Function 'ServeHTTP' has too many statements
      - path: memmetrics/ratio_test.go # TODO(ldez) must be fixed
        text: 'float-compare: use assert\.InEpsilon \(or InDelta\)'
      - path: memmetrics/roundtrip_test.go # TODO(ldez) must be fixed
        text: 'float-compare: use assert\.InEpsilon \(or InDelta\)'
      - path: memmetrics/anomaly_test.go # TODO(ldez) must be fixed
        text: 'float-compare: use assert\.InEpsilon \(or InDelta\)'
      - path: (.+)\.go$ # TODO(ldez) must be fixed
        text: 'SA1019: http.CloseNotifier has been deprecated'
      - path: (.+)\.go$ # TODO(ldez) must be fixed
        text: 'exported: func name will be used as roundrobin.RoundRobinRequestRewriteListener by other packages'
    paths:
      - internal/holsterv4

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