misspell: set ignore-words to lowercase

ldez · ldez · commit b9bd12676c77 · 2023-06-18T21:43:23.000+02:00
diff --git a/pkg/golinters/misspell.go b/pkg/golinters/misspell.go
@@ -95,7 +95,11 @@ func createMisspellReplacer(settings *config.MisspellSettings) (*misspell.Replac
 	}
 
 	if len(settings.IgnoreWords) != 0 {
-		replacer.RemoveRule(settings.IgnoreWords)
+		var ignoreWords []string
+		for _, word := range settings.IgnoreWords {
+			ignoreWords = append(ignoreWords, strings.ToLower(word))
+		}
+		replacer.RemoveRule(ignoreWords)
 	}
 
 	// It can panic.
diff --git a/test/testdata/configs/misspell.yml b/test/testdata/configs/misspell.yml
@@ -1,4 +1,6 @@
 linters-settings:
   misspell:
+    locale: US
     ignore-words:
       - langauge
+      - Dialogue
diff --git a/test/testdata/misspell.go b/test/testdata/misspell.go
@@ -7,3 +7,4 @@ func Misspell() {
 }
 
 // the word langauge should be ignored here: it's set in config
+// the word Dialogue should be ignored here: it's set in config

Original file line number	Diff line number	Diff line change
`@@ -95,7 +95,11 @@ func createMisspellReplacer(settings config.MisspellSettings) (misspell.Replac`
`95`	`95`	`}`
`96`	`96`
`97`	`97`	`if len(settings.IgnoreWords) != 0 {`
`98`		`- replacer.RemoveRule(settings.IgnoreWords)`
	`98`	`+ var ignoreWords []string`
	`99`	`+ for _, word := range settings.IgnoreWords {`
	`100`	`+ ignoreWords = append(ignoreWords, strings.ToLower(word))`
	`101`	`+ }`
	`102`	`+ replacer.RemoveRule(ignoreWords)`
`99`	`103`	`}`
`100`	`104`
`101`	`105`	`// It can panic.`
Original file line number	Diff line number	Diff line change
`@@ -7,3 +7,4 @@ func Misspell() {`
`7`	`7`	`}`
`8`	`8`
`9`	`9`	`// the word langauge should be ignored here: it's set in config`
	`10`	`+// the word Dialogue should be ignored here: it's set in config`