Minor improvements

analyzer.go

@@ -57,25 +57,31 @@ func run(pass *analysis.Pass) (interface{}, error) {
 	if conf.IsCheckEnabled(checks.ArgumentCheck) {
 		checker = append(checker, checks.NewArgumentAnalyzer(pass, conf))
 	}
+
 	if conf.IsCheckEnabled(checks.CaseCheck) {
 		checker = append(checker, checks.NewCaseAnalyzer(pass, conf))
 	}
+
 	if conf.IsCheckEnabled(checks.ConditionCheck) {
 		checker = append(checker, checks.NewConditionAnalyzer(pass, conf))
 	}
+
 	if conf.IsCheckEnabled(checks.OperationCheck) {
 		checker = append(checker, checks.NewOperationAnalyzer(pass, conf))
 	}
+
 	if conf.IsCheckEnabled(checks.ReturnCheck) {
 		checker = append(checker, checks.NewReturnAnalyzer(pass, conf))
 	}
+
 	if conf.IsCheckEnabled(checks.AssignCheck) {
 		checker = append(checker, checks.NewAssignAnalyzer(pass, conf))
 	}
 
 	i := pass.ResultOf[inspect.Analyzer].(*inspector.Inspector)
 
 	for _, c := range checker {
+		c := c
 		i.Preorder(c.NodeFilter(), func(node ast.Node) {
 			for _, exclude := range conf.Excludes {
 				if exclude.MatchString(pass.Fset.Position(node.Pos()).Filename) {

cmd/mnd/main.go

@@ -1,8 +1,9 @@
 package main
 
 import (
-	"github.com/tommy-muehle/go-mnd"
 	"golang.org/x/tools/go/analysis/singlechecker"
+
+	mnd "github.com/tommy-muehle/go-mnd"
 )
 
-func main() { singlechecker.Main(magic_numbers.Analyzer) }
+func main() { singlechecker.Main(mnd.Analyzer) }

config/config.go

@@ -28,9 +28,11 @@ func DefaultConfig() *Config {
 
 func WithOptions(options ...Option) *Config {
 	c := DefaultConfig()
+
 	for _, option := range options {
 		option(c)
 	}
+
 	return c
 }
 

config/config_test.go

@@ -1,16 +1,19 @@
-package config
+package config_test
 
 import (
+	"regexp"
 	"testing"
 
 	"github.com/stretchr/testify/assert"
+
+	"github.com/tommy-muehle/go-mnd/config"
 )
 
 func TestWithCustomChecks(t *testing.T) {
 	assert := assert.New(t)
 
-	c := WithOptions(
-		WithCustomChecks("return,operation"),
+	c := config.WithOptions(
+		config.WithCustomChecks("return,operation"),
 	)
 
 	assert.True(c.IsCheckEnabled("return"))
@@ -25,14 +28,14 @@ func TestWithCustomChecks(t *testing.T) {
 func TestZeroIsIgnoredNumber(t *testing.T) {
 	assert := assert.New(t)
 
-	assert.True(DefaultConfig().IsIgnoredNumber("0"))
+	assert.True(config.DefaultConfig().IsIgnoredNumber("0"))
 }
 
 func TestCanIgnoreCustomNumbers(t *testing.T) {
 	assert := assert.New(t)
 
-	c := WithOptions(
-		WithIgnoredNumbers("1,1000"),
+	c := config.WithOptions(
+		config.WithIgnoredNumbers("1,1000"),
 	)
 
 	assert.True(c.IsIgnoredNumber("0"))
@@ -42,3 +45,13 @@ func TestCanIgnoreCustomNumbers(t *testing.T) {
 	assert.False(c.IsIgnoredNumber("2"))
 	assert.False(c.IsIgnoredNumber("999"))
 }
+
+func TestWithExcludes(t *testing.T) {
+	assert := assert.New(t)
+
+	c := config.WithOptions(
+		config.WithExcludes(".*"),
+	)
+
+	assert.Contains(c.Excludes, regexp.MustCompile(".*"))
+}

tools/printer/main.go

@@ -25,6 +25,7 @@ var (
 
 func main() {
 	fset := token.NewFileSet()
+
 	f, err := parser.ParseFile(fset, "", src, 0)
 	if err != nil {
 		panic(err)