Merge pull request #63 from dfreilich/patch-2

Joe Burianek · web-flow · commit 2c6e83df1d2b · 2021-11-18T17:12:58.000-06:00
Avoid writing error to Stdout
diff --git a/autotag/main.go b/autotag/main.go
@@ -50,15 +50,17 @@ func main() {
 	})
 
 	if err != nil {
-		fmt.Println("Error initializing: ", err)
+		log.SetOutput(os.Stderr)
+		log.Println("Error initializing: " + err.Error())
 		os.Exit(1)
 	}
 
 	// Tag unless asked otherwise
 	if !opts.JustVersion {
 		err = r.AutoTag()
 		if err != nil {
-			fmt.Println("Error auto updating version: ", err.Error())
+			log.SetOutput(os.Stderr)
+			log.Println("Error auto updating version: " + err.Error())
 			os.Exit(1)
 		}
 	}

Original file line number	Diff line number	Diff line change
`@@ -50,15 +50,17 @@ func main() {`
`50`	`50`	`})`
`51`	`51`
`52`	`52`	`if err != nil {`
`53`		`- fmt.Println("Error initializing: ", err)`
	`53`	`+ log.SetOutput(os.Stderr)`
	`54`	`+ log.Println("Error initializing: " + err.Error())`
`54`	`55`	`os.Exit(1)`
`55`	`56`	`}`
`56`	`57`
`57`	`58`	`// Tag unless asked otherwise`
`58`	`59`	`if !opts.JustVersion {`
`59`	`60`	`err = r.AutoTag()`
`60`	`61`	`if err != nil {`
`61`		`- fmt.Println("Error auto updating version: ", err.Error())`
	`62`	`+ log.SetOutput(os.Stderr)`
	`63`	`+ log.Println("Error auto updating version: " + err.Error())`
`62`	`64`	`os.Exit(1)`
`63`	`65`	`}`
`64`	`66`	`}`