summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLuke Shumaker <lukeshu@lukeshu.com>2023-01-01 22:23:10 -0700
committerLuke Shumaker <lukeshu@lukeshu.com>2023-01-01 22:42:08 -0700
commit162636379372d081ee91dd7bd003f2f215e1a856 (patch)
tree0ae6ea95151af2e4823b97f72ee5debcd102a2d0
parent9fc19f6292636c9c62acdea2342bba271dae182e (diff)
.golangci.yml: Organize disabled linters
-rw-r--r--.golangci.yml34
1 files 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: