add error handling

denis-tingaikin · denis-tingaikin · commit daf4b8d2c672 · 2025-08-03T01:25:19.000+03:00
Signed-off-by: Denis Tingaikin &lt;denis.tingaikin@outlook.com&gt;
diff --git a/pkg/golinters/goheader/goheader.go b/pkg/golinters/goheader/goheader.go
@@ -8,6 +8,7 @@ import (
 
 	"github.com/golangci/golangci-lint/v2/pkg/config"
 	"github.com/golangci/golangci-lint/v2/pkg/goanalysis"
+	"github.com/golangci/golangci-lint/v2/pkg/golinters/internal"
 )
 
 const linterName = "goheader"
@@ -22,7 +23,9 @@ func New(settings *config.GoHeaderSettings, replacer *strings.Replacer) *goanaly
 		}
 	}
 	var goheaderSettings goheader.Settings
-	_ = conf.FillSettings(&goheaderSettings)
+	if err := conf.FillSettings(&goheaderSettings); err != nil {
+		internal.LinterLogger.Fatalf("%s: invalid toolchain pattern: %s", linterName, err.Error())
+	}
 	return goanalysis.NewLinter(
 		linterName,
 		"Checks if file header matches to pattern",

Original file line number	Diff line number	Diff line change
`@@ -8,6 +8,7 @@ import (`
`8`	`8`
`9`	`9`	`"github.com/golangci/golangci-lint/v2/pkg/config"`
`10`	`10`	`"github.com/golangci/golangci-lint/v2/pkg/goanalysis"`
	`11`	`+ "github.com/golangci/golangci-lint/v2/pkg/golinters/internal"`
`11`	`12`	`)`
`12`	`13`
`13`	`14`	`const linterName = "goheader"`
`@@ -22,7 +23,9 @@ func New(settings config.GoHeaderSettings, replacer strings.Replacer) *goanaly`
`22`	`23`	`}`
`23`	`24`	`}`
`24`	`25`	`var goheaderSettings goheader.Settings`
`25`		`- _ = conf.FillSettings(&goheaderSettings)`
	`26`	`+ if err := conf.FillSettings(&goheaderSettings); err != nil {`
	`27`	`+ internal.LinterLogger.Fatalf("%s: invalid toolchain pattern: %s", linterName, err.Error())`
	`28`	`+ }`
`26`	`29`	`return goanalysis.NewLinter(`
`27`	`30`	`linterName,`
`28`	`31`	`"Checks if file header matches to pattern",`