chore(refactor): remove boolean flag

Rather than pass a boolean to result.publish(), I now switch on that
same boolean to call two different methods, publish() or publishError().

On the one hand, this feels very picky. On the other hand, why not?

internal/cli/clone.go

@@ -30,7 +30,12 @@ func (app *appEnv) clone(rs []Repo) {
 	}
 	for range rs {
 		res := <-ch
-		res.publish(app.quiet)
+		switch app.quiet {
+		case true:
+			res.publishError()
+		default:
+			res.publish()
+		}
 	}
 }
 

internal/cli/result.go

@@ -10,15 +10,16 @@ type result struct {
 	isErr bool
 }
 
-func (r result) publish(quiet bool) {
+func (r result) publish() {
 	if r.isErr {
 		fmt.Fprintln(os.Stderr, r.msg)
 		return
 	}
+	fmt.Fprintln(os.Stdout, r.msg)
+}
 
-	if quiet {
-		return
+func (r result) publishError() {
+	if r.isErr {
+		fmt.Fprintln(os.Stderr, r.msg)
 	}
-
-	fmt.Fprintln(os.Stdout, r.msg)
 }

internal/cli/update.go

@@ -25,7 +25,12 @@ func (app *appEnv) update(rs []Repo) {
 	}
 	for range rs {
 		res := <-ch
-		res.publish(app.quiet)
+		switch app.quiet {
+		case true:
+			res.publishError()
+		default:
+			res.publish()
+		}
 	}
 }