pr status: avoid printing a lonely "1" when there is only one Check

mislav · mislav · commit d1eac7b211b7 · 2019-11-15T12:35:44.000+01:00
In a repository that only has a single Check configured (e.g. this
repo), we would print "checks: 1" for PRs where the CI is passing. This
looks akward when repeated for each PR and provides little useful
information.

This avoids ever printing "1" and instead prints "failing", "pending",
or "success", respectively. We now only show numbers for repositories
that have more than one Check runs.
diff --git a/command/pr.go b/command/pr.go
@@ -258,7 +258,7 @@ func printPrs(prs ...api.PullRequest) {
 			fmt.Printf("\n  ")
 		}
 
-		if checks.Total > 0 {
+		if checks.Total > 1 {
 			var ratio string
 			if checks.Failing > 0 {
 				ratio = fmt.Sprintf("%d/%d", checks.Passing, checks.Total)
@@ -271,6 +271,16 @@ func printPrs(prs ...api.PullRequest) {
 				ratio = utils.Green(ratio)
 			}
 			fmt.Printf(" - checks: %s", ratio)
+		} else if checks.Total == 1 {
+			var state string
+			if checks.Failing > 0 {
+				state = utils.Red("failing")
+			} else if checks.Pending > 0 {
+				state = utils.Yellow("pending")
+			} else if checks.Passing == checks.Total {
+				state = utils.Green("success")
+			}
+			fmt.Printf(" - checks: %s", state)
 		}
 
 		if reviews.ChangesRequested {
