sav

Author: AskAlexSharov

.golangci.yml

@@ -1,65 +1,85 @@
-# This file configures github.com/golangci/golangci-lint.
-
+version: "2"
 run:
-  timeout: 10m
   tests: true
-
 linters:
-  presets:
-    - bugs
-    - error
-    - unused
-    - performance
+  enable:
+    - asasalint
+    - asciicheck
+    - bidichk
+    - bodyclose
+    - durationcheck
+    - fatcontext
+    - gochecksumtype
+    - gocritic
+    - gosmopolitan
+    - loggercheck
+    - nilnesserr
+    - perfsprint
+    - prealloc
+    - reassign
+    - rowserrcheck
+    - spancheck
+    - sqlclosecheck
+    - testifylint
+    - unconvert
+    - wastedassign
+    - zerologlint
   disable:
-    - gosec # enable it after 1-2 weeks. it failing CI withoutt enough info to fix: https://github.com/erigontech/erigon/actions/runs/5928644129/job/16074701625
-    - exhaustive
-    - musttag
-    - contextcheck
-    - wrapcheck
-    - err113
-    - unparam
-    - errchkjson #TODO: enable me
-    - makezero #TODO: enable me
+    - errorlint #TODO: enable me
     - noctx #TODO: enable me
     - nilerr #TODO: enable me
-    - errorlint #TODO: enable me
     - unused #TODO: enable me
+    - makezero #TODO: enable me
+    - contextcheck
+    - err113
+    - errchkjson
+    - exhaustive
     - gocheckcompilerdirectives
+    - gosec
+    - musttag
     - protogetter
-    - recvcheck #TODO: enable me
+    - recvcheck
+    - unparam
+    - wrapcheck
+  settings:
+    goconst:
+      min-len: 3
+      min-occurrences: 6
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    rules:
+      - linters:
+          - gosec
+          - unparam
+          - unused
+        path: _test\.go
+      - linters:
+          - golint
+        text: should be
+      - linters:
+          - err113
+        text: do not define dynamic errors
+      - linters:
+          - staticcheck
+        text: (should be|should have name of the form)
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
   enable:
-    - unconvert
-    #    - predeclared #TODO: enable me
-    #    - thelper #TODO: enable me
-    - testifylint #TODO: enable me
-    - wastedassign
     - gofmt
-    - gocritic
-    - perfsprint
-#    - revive
-#    - forcetypeassert
-#    - stylecheck
-
-linters-settings:
-  gofmt:
-    simplify: true
-  goconst:
-    min-len: 3 # minimum length of string constant
-    min-occurrences: 6 # minimum number of occurrences
-
-issues:
-  exclude-rules:
-    - path: _test\.go
-      linters:
-        - gosec
-        - unused
-        - unparam
-    - linters:
-        - golint
-      text: "should be"
-    - linters:
-        - err113
-      text: "do not define dynamic errors"
-    - linters:
-        - stylecheck
-      text: "(should be|should have name of the form)"
+  settings:
+    gofmt:
+      simplify: true
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$