quickfix: error logged needlessly, checking for sentinel error value

nilslice · nilslice · commit eae15e37ed98 · 2019-03-26T14:55:23.000-06:00
diff --git a/cmd/protolock/main.go b/cmd/protolock/main.go
@@ -121,14 +121,14 @@ func status(cfg *protolock.Config) {
 	if *plugins != "" {
 		report, err = runPlugins(*plugins, report)
 		if err != nil {
-			fmt.Println(err)
+			fmt.Println("[protolock]:", err)
 			os.Exit(1)
 		}
 	}
 
 	code, err := protolock.HandleReport(report, os.Stdout, err)
-	if err != nil {
-		fmt.Println(err)
+	if err != protolock.ErrWarningsFound && err != nil {
+		fmt.Println("[protolock]:", err)
 		os.Exit(1)
 	}
 

Original file line number	Diff line number	Diff line change
`@@ -121,14 +121,14 @@ func status(cfg *protolock.Config) {`
`121`	`121`	`if *plugins != "" {`
`122`	`122`	`report, err = runPlugins(*plugins, report)`
`123`	`123`	`if err != nil {`
`124`		`- fmt.Println(err)`
	`124`	`+ fmt.Println("[protolock]:", err)`
`125`	`125`	`os.Exit(1)`
`126`	`126`	`}`
`127`	`127`	`}`
`128`	`128`
`129`	`129`	`code, err := protolock.HandleReport(report, os.Stdout, err)`
`130`		`- if err != nil {`
`131`		`- fmt.Println(err)`
	`130`	`+ if err != protolock.ErrWarningsFound && err != nil {`
	`131`	`+ fmt.Println("[protolock]:", err)`
`132`	`132`	`os.Exit(1)`
`133`	`133`	`}`
`134`	`134`