Enable more Go linters and document exclusions

Author: cbandy

.golangci.bck.yaml

@@ -1,44 +1,86 @@
-# https://golangci-lint.run/usage/configuration/
+# https://golangci-lint.run/usage/configuration
 #
 # This file is for linters that might be interesting to enforce in the future.
 # Rules that should be enforced immediately belong in [.golangci.yaml].
 #
 # Both files are used by [.github/workflows/lint.yaml].
+version: "2"
 
+# https://golangci-lint.run/usage/linters
 linters:
-  disable-all: true
-  enable:
-    - contextcheck
-    - err113
-    - gocritic
-    - godot
-    - godox
-    - gofumpt
-    - gosec # exclude-use-default
-    - nilnil
+  default: all
+  disable:
+    - asasalint
+    - asciicheck
+    - bidichk
+    - bodyclose
+    - copyloopvar
+    - depguard
+    - dupword
+    - durationcheck
+    - errchkjson
+    - errname
+    - errorlint
+    - exhaustive
+    - exptostd
+    - fatcontext
+    - forbidigo
+    - ginkgolinter
+    - gocheckcompilerdirectives
+    - gochecksumtype
+    - goheader
+    - gomoddirectives
+    - gomodguard
+    - goprintffuncname
+    - gosec
+    - gosmopolitan
+    - grouper
+    - iface
+    - importas
+    - interfacebloat
+    - intrange
+    - loggercheck
+    - makezero
+    - mirror
+    - misspell
+    - musttag
+    - nilerr
+    - nilnesserr
+    - noctx
     - nolintlint
+    - nosprintfhostport
+    - prealloc
     - predeclared
-    - revive
-    - staticcheck # exclude-use-default
-    - tenv
-    - thelper
-    - tparallel
+    - promlinter
+    - protogetter
+    - reassign
+    - recvcheck
+    - rowserrcheck
+    - sloglint
+    - spancheck
+    - sqlclosecheck
+    - tagalign
+    - testifylint
+    - unconvert
+    - unparam
+    - usestdlibvars
+    - usetesting
     - wastedassign
-  presets:
-    - performance
+    - wsl
+    - zerologlint
 
-issues:
-  exclude-rules:
-    # We call external linters when they are installed: Flake8, ShellCheck, etc.
-    - linters: [gosec]
-      path: '_test[.]go$'
-      text: 'G204: Subprocess launched with variable'
+  settings:
+    thelper:
+      # https://github.com/kulti/thelper/issues/27
+      tb:   { begin: true, first: true }
+      test: { begin: true, first: true, name: true }
 
-  # https://github.com/golangci/golangci-lint/issues/2239
-  exclude-use-default: false
-
-linters-settings:
-  thelper:
-    # https://github.com/kulti/thelper/issues/27
-    tb:   { begin: true, first: true }
-    test: { begin: true, first: true, name: true }
+  exclusions:
+    warn-unused: true
+    # Ignore built-in exclusions
+    presets: []
+    rules:
+      # We call external linters when they are installed: Flake8, ShellCheck, etc.
+      - linters: [gosec]
+        path: '_test[.]go$'
+        text: 'G204: Subprocess launched with variable'