Fix checker set

Some of the enabled checkers were deprecated, had known issues, or
returned erroneous results.  They've been commented out, with comments
as to why they're not run.

Author: DirectXMan12

test.sh

@@ -115,23 +115,23 @@ header_text "running golangci-lint"
 
 golangci-lint run --disable-all \
     --enable=misspell \
-    --enable=structcheck \
     --enable=golint \
     --enable=deadcode \
     --enable=goimports \
     --enable=errcheck \
     --enable=varcheck \
-    --enable=goconst \
     --enable=unparam \
     --enable=ineffassign \
     --enable=nakedret \
-    --enable=interfacer \
     --enable=misspell \
     --enable=gocyclo \
     --enable=gosec \
-    --skip-dirs=parse \
     ./pkg/... ./cmd/...
 
+# --enable=structcheck \  # doesn't understand embedded structs
+# --enable=goconst \  # complains about constants that shouldn't be constants
+# --enable=interfacer \ # just kinda strange, deprecated, has bad suggestions that defeat self-documenting code
+
 header_text "running go test"
 
 go test -mod=vendor ./pkg/... ./cmd/... -parallel 4