Merge pull request #6 from karamaru-alpha/fix/karamaru/quit-init

refactor: remove init function

cmd/copyloopvar/main.go

@@ -6,4 +6,4 @@ import (
 	"github.com/karamaru-alpha/copyloopvar"
 )
 
-func main() { singlechecker.Main(copyloopvar.Analyzer) }
+func main() { singlechecker.Main(copyloopvar.NewAnalyzer()) }

copyloopvar.go

@@ -10,21 +10,19 @@ import (
 	"golang.org/x/tools/go/ast/inspector"
 )
 
-var Analyzer = &analysis.Analyzer{
-	Name: "copyloopvar",
-	Doc:  "copyloopvar is a linter detects places where loop variables are copied",
-	Run:  run,
-	Requires: []*analysis.Analyzer{
-		inspect.Analyzer,
-	},
-}
-
-var (
-	ignoreAlias bool
-)
+var ignoreAlias bool
 
-func init() {
-	Analyzer.Flags.BoolVar(&ignoreAlias, "ignore-alias", false, "ignore aliasing of loop variables")
+func NewAnalyzer() *analysis.Analyzer {
+	analyzer := &analysis.Analyzer{
+		Name: "copyloopvar",
+		Doc:  "copyloopvar is a linter detects places where loop variables are copied",
+		Run:  run,
+		Requires: []*analysis.Analyzer{
+			inspect.Analyzer,
+		},
+	}
+	analyzer.Flags.BoolVar(&ignoreAlias, "ignore-alias", false, "ignore aliasing of loop variables")
+	return analyzer
 }
 
 func run(pass *analysis.Pass) (any, error) {

copyloopvar_test.go

@@ -10,15 +10,15 @@ import (
 func TestAnalyzer(t *testing.T) {
 	t.Run("basic", func(t *testing.T) {
 		testdata := testutil.WithModules(t, analysistest.TestData(), nil)
-		analysistest.Run(t, testdata, Analyzer, "basic")
+		analysistest.Run(t, testdata, NewAnalyzer(), "basic")
 	})
 
 	t.Run("ignore-alias", func(t *testing.T) {
-		err := Analyzer.Flags.Set("ignore-alias", "true")
-		if err != nil {
+		analyzer := NewAnalyzer()
+		if err := analyzer.Flags.Set("ignore-alias", "true"); err != nil {
 			t.Error(err)
 		}
 		testdata := testutil.WithModules(t, analysistest.TestData(), nil)
-		analysistest.Run(t, testdata, Analyzer, "ignorealias")
+		analysistest.Run(t, testdata, analyzer, "ignorealias")
 	})
 }