Use stderr for progress information

Like this blog post describes, it's a unix convention: https://monkey.org/~marius/unix-tools-hints.html Signed-off-by: Artem Sidorenko <[email protected]>
artem-sidorenko · Mar 20, 2019 · 4a723a3 · 4a723a3
1 parent 8f39ee8
commit 4a723a3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/cli/commands/generate/generate.go b/cli/commands/generate/generate.go
@@ -39,8 +39,8 @@ import (
 
 // Stdout references the Stdout writer for generate command
 var Stdout io.Writer = os.Stdout // nolint: gochecknoglobals
-// ProgressStdout references the Stdout writer for progress information
-var ProgressStdout io.Writer = os.Stdout // nolint: gochecknoglobals
+// ProgressWriter references the writer for progress information
+var ProgressWriter io.Writer = os.Stderr // nolint: gochecknoglobals
 
 // Generate implements the CLI subcommand generate
 func Generate(ctx *cli.Context) error { // nolint: gocyclo
@@ -210,7 +210,7 @@ func getConnectorData(
 
 	// invoke the progress printer
 	printProgress(
-		ctx, ProgressStdout,
+		ctx, ProgressWriter,
 		ctagscounter, cmaxtags,
 		cissuescounter, cmaxissues,
 		cmrscounter, cmaxmrs)

diff --git a/cli/commands/generate/generate_test.go b/cli/commands/generate/generate_test.go
@@ -252,7 +252,7 @@ func TestGenerate(t *testing.T) { // nolint: gocyclo
 
 		// avoid progress output
 		progressOutput := &bytes.Buffer{}
-		generate.ProgressStdout = progressOutput
+		generate.ProgressWriter = progressOutput
 
 		testconnector.RetTestingTag = true
 		testconnector.RepositoryExistsFail = tt.repositoryExistsFail