Merge pull request #10 from uudashr/fix-debugflag-coflict

Use nerd flag to avoid flag conflict

Author: uudashr

.github/workflows/go.yml

@@ -35,6 +35,6 @@ jobs:
     - uses: actions/setup-go@v5
       with:
         go-version: oldstable
-    - uses: golangci/golangci-lint-action@v6
+    - uses: golangci/golangci-lint-action@v8
       with:
-        version: latest
+        version: v2.1

.golangci.yml

@@ -1,11 +1,6 @@
+version: "2"
 linters:
   enable:
-    - errcheck
-    - gosimple
-    - govet
-    - ineffassign
-    - staticcheck
-    - unused
     - asasalint
     - asciicheck
     - bidichk
@@ -24,7 +19,6 @@ linters:
     - errorlint
     - exhaustive
     - fatcontext
-    - gci
     - ginkgolinter
     - gocheckcompilerdirectives
     - gochecknoinits
@@ -33,10 +27,7 @@ linters:
     - gocritic
     - godot
     - godox
-    - gofmt
-    - gofumpt
     - goheader
-    - goimports
     - gomoddirectives
     - gomodguard
     - goprintffuncname
@@ -73,7 +64,6 @@ linters:
     - spancheck
     - tagalign
     - tagliatelle
-    - tenv
     - testableexamples
     - testifylint
     - testpackage
@@ -85,18 +75,40 @@ linters:
     - whitespace
     - wsl
     - zerologlint
-
-linters-settings:
-  varnamelen:
-    ignore-type-assert-ok: true
-    ignore-decls:
-      - i int
-      - tx *sql.Tx
-      - ts *ast.TypeSpec
-      - fn *ast.FuncDecl
-      - fun *ast.FuncDecl
-      - fnDecl *ast.FuncDecl
-      - funDecl *ast.FuncDecl
-  mnd:
-    ignored-numbers:
-      - "2"
+  settings:
+    mnd:
+      ignored-numbers:
+        - "2"
+    varnamelen:
+      ignore-type-assert-ok: true
+      ignore-decls:
+        - i int
+        - tx *sql.Tx
+        - ts *ast.TypeSpec
+        - fn *ast.FuncDecl
+        - fun *ast.FuncDecl
+        - fnDecl *ast.FuncDecl
+        - funDecl *ast.FuncDecl
+  exclusions:
+    generated: lax
+    presets:
+      - comments
+      - common-false-positives
+      - legacy
+      - std-error-handling
+    paths:
+      - third_party$
+      - builtin$
+      - examples$
+formatters:
+  enable:
+    - gci
+    - gofmt
+    - gofumpt
+    - goimports
+  exclusions:
+    generated: lax
+    paths:
+      - third_party$
+      - builtin$
+      - examples$

identical/identical.go

@@ -27,7 +27,7 @@ func newAnalyzer() *analysis.Analyzer {
 		Run:      r.run,
 	}
 
-	analyzer.Flags.BoolVar(&r.debug, "debug", false, "enable debug mode")
+	analyzer.Flags.BoolVar(&r.debug, "nerd", false, "enable nerd mode")
 
 	return analyzer
 }

opaque/opaque.go

@@ -27,7 +27,7 @@ func newAnalyzer() *analysis.Analyzer {
 		Run:      r.run,
 	}
 
-	analyzer.Flags.BoolVar(&r.debug, "debug", false, "enable debug mode")
+	analyzer.Flags.BoolVar(&r.debug, "nerd", false, "enable nerd mode")
 
 	return analyzer
 }

unused/unused.go

@@ -28,7 +28,7 @@ func newAnalyzer() *analysis.Analyzer {
 		Run:      r.run,
 	}
 
-	analyzer.Flags.BoolVar(&r.debug, "debug", false, "enable debug mode")
+	analyzer.Flags.BoolVar(&r.debug, "nerd", false, "enable nerd mode")
 	analyzer.Flags.StringVar(&r.exclude, "exclude", "", "comma-separated list of packages to exclude from the check")
 
 	return analyzer
@@ -40,6 +40,8 @@ type runner struct {
 }
 
 func (r *runner) run(pass *analysis.Pass) (interface{}, error) {
+	fmt.Println("Running unused interface analyzer")
+
 	excludes := strings.Split(r.exclude, ",")
 	if slices.Contains(excludes, pass.Pkg.Path()) {
 		return nil, nil