Merge pull request #306 from broady/patch-1

Don't use log.Fatal for printing usage.

Author: kytrinyx

cmd/download.go

@@ -23,8 +23,8 @@ func Download(ctx *cli.Context) {
 
 	args := ctx.Args()
 	if len(args) != 1 {
-		msg := "Usage: exercism download SUBMISSION_ID"
-		log.Fatal(msg)
+		fmt.Fprintf(os.Stderr, "Usage: exercism download SUBMISSION_ID")
+		os.Exit(1)
 	}
 
 	submission, err := client.Download(args[0])

cmd/list.go

@@ -3,6 +3,7 @@ package cmd
 import (
 	"fmt"
 	"log"
+	"os"
 
 	"github.com/codegangsta/cli"
 	"github.com/exercism/cli/api"
@@ -20,8 +21,8 @@ func List(ctx *cli.Context) {
 	args := ctx.Args()
 
 	if len(args) != 1 {
-		msg := "Usage: exercism list TRACK_ID"
-		log.Fatal(msg)
+		fmt.Fprintf(os.Stderr, "Usage: exercism list TRACK_ID")
+		os.Exit(1)
 	}
 
 	trackID := args[0]

cmd/open.go

@@ -1,7 +1,9 @@
 package cmd
 
 import (
+	"fmt"
 	"log"
+	"os"
 	"os/exec"
 	"runtime"
 	"strings"
@@ -21,8 +23,8 @@ func Open(ctx *cli.Context) {
 
 	args := ctx.Args()
 	if len(args) != 2 {
-		msg := "Usage: exercism open TRACK_ID PROBLEM"
-		log.Fatal(msg)
+		fmt.Fprintf(os.Stderr, "Usage: exercism open TRACK_ID PROBLEM")
+		os.Exit(1)
 	}
 
 	trackID := args[0]

cmd/skip.go

@@ -3,6 +3,7 @@ package cmd
 import (
 	"fmt"
 	"log"
+	"os"
 
 	"github.com/codegangsta/cli"
 	"github.com/exercism/cli/api"
@@ -18,8 +19,8 @@ func Skip(ctx *cli.Context) {
 	args := ctx.Args()
 
 	if len(args) != 2 {
-		msg := "Usage: exercism skip TRACK_ID PROBLEM"
-		log.Fatal(msg)
+		fmt.Fprintf(os.Stderr, "Usage: exercism skip TRACK_ID PROBLEM")
+		os.Exit(1)
 	}
 
 	var (

cmd/status.go

@@ -3,6 +3,7 @@ package cmd
 import (
 	"fmt"
 	"log"
+	"os"
 
 	"github.com/codegangsta/cli"
 	"github.com/exercism/cli/api"
@@ -19,7 +20,8 @@ func Status(ctx *cli.Context) {
 	args := ctx.Args()
 
 	if len(args) != 1 {
-		log.Fatal("Usage: exercism status TRACK_ID")
+		fmt.Fprintf(os.Stderr, "Usage: exercism status TRACK_ID")
+		os.Exit(1)
 	}
 
 	client := api.NewClient(c)