From 162636379372d081ee91dd7bd003f2f215e1a856 Mon Sep 17 00:00:00 2001
From: Luke Shumaker <lukeshu@lukeshu.com>
Date: Sun, 1 Jan 2023 22:23:10 -0700
Subject: .golangci.yml: Organize disabled linters

---
 .golangci.yml | 34 ++++++++++++++++++++--------------
 1 file changed, 20 insertions(+), 14 deletions(-)

diff --git a/.golangci.yml b/.golangci.yml
index c27b8d4..d1c1798 100644
--- a/.golangci.yml
+++ b/.golangci.yml
@@ -21,33 +21,39 @@ linters:
     - rowserrcheck
     - sqlclosecheck
     - wastedassign
+    - ireturn # golangci-lint doesn't claim it doesn't, but it doesn't
 
-    # These are disabled not because I think they're bad, but because
-    # they currently don't pass, and I haven't evaluated them yet.
+    # Style
+    - godot
+    - lll
+    - nlreturn
+    - nonamedreturns # I name returns for godoc purposes.
+    - wsl
+
+    # Complexity; sometimes code is just complex.
     - cyclop
-    - exhaustive
-    - exhaustruct
     - funlen
-    - gochecknoglobals
-    - gochecknoinits
     - gocognit
     - gocyclo
-    - godot
-    - godox
-    - goerr113
-    - ireturn
-    - lll
     - maintidx
     - nestif
-    - nlreturn
-    - nonamedreturns
+
+    # Miscellaneous
+    - goerr113  # forbids fmt.Errorf(%w), which is just silly
+    - godox # I'm OK checking in to-be-completed tasks
+
+    # These are disabled not because I think they're bad, but because
+    # they currently don't pass, and I haven't really evaluated them yet.
+    - exhaustive
+    - exhaustruct
+    - gochecknoglobals
+    - gochecknoinits
     - revive
     - testpackage
     - thelper
     - unparam
     - varnamelen
     - wrapcheck
-    - wsl
 linters-settings:
   gci:
     sections:
-- 
cgit v1.2.3