lint/golangci-lint: update golangci-lint config

Author: zchee

.golangci.yml

@@ -1,46 +1,88 @@
 run:
-  concurrency: 12
-  deadline: 5m
   issues-exit-code: 1
-  tests: false
-  # skip-dirs:
+  tests: true
+  skip-dirs:
+    - "vendor$"
   skip-files:
     - ".*\\.pb\\.go"
-  modules-download-mode: vendor
+    - ".*(.|_)gen\\.go"
+  modules-download-mode: readonly
 
 linters-settings:
-  depguard:
-    list-type: blacklist
-    include-go-root: false
-    packages:
-      - github.com/davecgh/go-spew/spew
+  dupl:
+    threshold: 400
   errcheck:
     check-type-assertions: true
     check-blank: true
     exclude: .errcheckignore
+  funlen:
+    lines: 80
+    statements: 40
   goconst:
     min-len: 3
     min-occurrences: 3
-  gofmt:
-    simplify: true
-  goimports:
-    local-prefixes: github.com/go-language-server/jsonrpc2
-  golint:
-    min-confidence: 0.3
-  govet:
-    check-shadowing: false
   gocritic:
     enabled-tags:
       - diagnostic
-      - style
-      - performance
       - experimental
       - opinionated
+      - performance
+      - style
     settings:
-      captLocal:
-        paramsOnly: true
+      hugeParam:
+        sizeThreshold: 80
+      rangeExprCopy:
+        sizeThreshold: 512
       rangeValCopy:
-        sizeThreshold: 32
+        sizeThreshold: 128
+  gocyclo:
+    min-complexity: 10
+  gofmt:
+    simplify: true
+  goimports:
+    local-prefixes: github.com/go-language-server/jsonrpc2
+  golint:
+    min-confidence: 0.3
+  govet:
+    enable:
+      -  asmdecl
+      -  assign
+      -  atomic
+      -  atomicalign
+      -  bools
+      -  buildssa
+      -  buildtag
+      -  cgocall
+      -  composite
+      -  copylock
+      -  ctrlflow
+      -  deepequalerrors
+      -  errorsas
+      -  findcall
+      -  httpresponse
+      -  inspect
+      -  loopclosure
+      -  lostcancel
+      -  nilfunc
+      -  nilness
+      -  pkgfact
+      -  printf
+      -  shift
+      -  sortslice
+      -  stdmethods
+      -  structtag
+      -  tests
+      -  unmarshal
+      -  unreachable
+      -  unsafeptr
+      -  unusedresult
+    disable:
+      -  shadow
+  lll:
+    line-length: 180
+    tab-width: 1
+  maligned:
+    suggest-new: false
   misspell:
     locale: US
   nakedret:
@@ -53,45 +95,89 @@ linters-settings:
     algo: cha
     check-exported: true
   unused:
-    check-exported: true
+    check-exported: false
 
 linters:
+  # disabled:
+    # - funlen
+    # - gochecknoglobals
+    # - gochecknoinits
+    # - gocyclo
+    # - godox
+    # - gomnd
+    # - maligned
+    # - megacheck
+    # - scopelint
+    # - wsl
+  disable-all: true
   enable:
+    - bodyclose
     - deadcode
     - depguard
+    - dogsled
     - dupl
     - errcheck
+    - gocognit
     - goconst
     - gocritic
     - gofmt
     - goimports
     - golint
+    - gosec
+    - gosimple
+    - govet
     - ineffassign
-    - maligned
+    - interfacer
+    - lll
     - misspell
+    - nakedret
     - prealloc
+    - staticcheck
     - structcheck
+    - stylecheck
     - typecheck
     - unconvert
-    - varcheck
-    - gosimple
-    - govet
-    - interfacer
-    - staticcheck
-    - stylecheck
     - unparam
-  disable:
-    - gochecknoglobals
-    - gochecknoinits
-    - gocyclo
-    - gosec
-    - lll
-    - nakedret
     - unused
-  fast: true
+    - varcheck
+    - whitespace
 
 issues:
-  exclude-use-default: false
+  exclude-use-default: true
+  exclude-rules:
+    - path: _test\.go
+      linters:
+        - funlen
+        - lll
+        - errcheck
+    - path: jsonrpc2.go
+      linters:
+        - gocognit
+      text: "func `.*.Run`"
+    - path: jsonrpc2.go
+      linters:
+        - deadcode
+        - unused
+        - varcheck
+      text: "`requestWaiting` is unused"
+    - path: jsonrpc2.go
+      linters:
+        - deadcode
+        - unused
+        - varcheck
+      text: "`requestDone` is unused"
+    - path: errors.go
+      linters:
+        - deadcode
+        - unused
+        - varcheck
+      text: "`codeServerErrorStart` is unused"
+    - path: errors.go
+      linters:
+        - deadcode
+        - unused
+        - varcheck
+      text: "`codeServerErrorEnd` is unused"
 
 output:
   format: colored-line-number