feat: make fmt.FPrintln instead of klog while printing error message (#719)

developer-guy · web-flow · commit 21e4fc7d84f8 · 2021-10-14T01:25:19.000-07:00
Signed-off-by: Batuhan Apaydın &lt;batuhan.apaydin@trendyol.com&gt;
diff --git a/cmd/krew/cmd/root.go b/cmd/krew/cmd/root.go
@@ -76,7 +76,8 @@ func Execute() {
 		if klog.V(1).Enabled() {
 			klog.Fatalf("%+v", err) // with stack trace
 		} else {
-			klog.Fatal(err) // just error message
+			fmt.Fprintln(os.Stderr, err)
+			os.Exit(1)
 		}
 	}
 }

Original file line number	Diff line number	Diff line change
`@@ -76,7 +76,8 @@ func Execute() {`
`76`	`76`	`if klog.V(1).Enabled() {`
`77`	`77`	`klog.Fatalf("%+v", err) // with stack trace`
`78`	`78`	`} else {`
`79`		`- klog.Fatal(err) // just error message`
	`79`	`+ fmt.Fprintln(os.Stderr, err)`
	`80`	`+ os.Exit(1)`
`80`	`81`	`}`
`81`	`82`	`}`
`82`	`83`	`}`