Fix start span matchers slice flag (#20)

jjti · web-flow · commit 128847493f00 · 2024-04-22T19:16:02.000-04:00
diff --git a/cmd/spancheck/main.go b/cmd/spancheck/main.go
@@ -33,7 +33,9 @@ func main() {
 	cfg.EnabledChecks = strings.Split(checkStrings, ",")
 	cfg.IgnoreChecksSignaturesSlice = strings.Split(ignoreCheckSignatures, ",")
 
-	cfg.StartSpanMatchersSlice = append(cfg.StartSpanMatchersSlice, strings.Split(extraStartSpanSignatures, ",")...)
+	if extraStartSpanSignatures != "" {
+		cfg.StartSpanMatchersSlice = append(cfg.StartSpanMatchersSlice, strings.Split(extraStartSpanSignatures, ",")...)
+	}
 
 	singlechecker.Main(spancheck.NewAnalyzerWithConfig(cfg))
 }

Original file line number	Diff line number	Diff line change
`@@ -33,7 +33,9 @@ func main() {`
`33`	`33`	`cfg.EnabledChecks = strings.Split(checkStrings, ",")`
`34`	`34`	`cfg.IgnoreChecksSignaturesSlice = strings.Split(ignoreCheckSignatures, ",")`
`35`	`35`
`36`		`- cfg.StartSpanMatchersSlice = append(cfg.StartSpanMatchersSlice, strings.Split(extraStartSpanSignatures, ",")...)`
	`36`	`+ if extraStartSpanSignatures != "" {`
	`37`	`+ cfg.StartSpanMatchersSlice = append(cfg.StartSpanMatchersSlice, strings.Split(extraStartSpanSignatures, ",")...)`
	`38`	`+ }`
`37`	`39`
`38`	`40`	`singlechecker.Main(spancheck.NewAnalyzerWithConfig(cfg))`
`39`	`41`	`}`