mkideal / cli / 44

64%

Pull #45

travis-ci

Fix issue #43

Introduce an integer return value for `func Run(...)` in cli.go.
This integer can be passed to `os.Exit` as recommended by all examples.

  % git diff -u cli.go
  diff --git a/cli.go b/cli.go
  index 41ce39e..64fb55c 100644
  --- a/cli.go
  +++ b/cli.go
  @@ -11,12 +11,12 @@ import (
   )

   // Run runs a single command app
  -func Run(argv interface{}, fn CommandFunc, descs ...string) {
  -       RunWithArgs(argv, os.Args, fn, descs...)
  +func Run(argv interface{}, fn CommandFunc, descs ...string) int {
  +       return RunWithArgs(argv, os.Args, fn, descs...)
   }

   // RunWithArgs is similar to Run, but with args instead of os.Args
  -func RunWithArgs(argv interface{}, args []string, fn CommandFunc, descs
  ...string) {
  +func RunWithArgs(argv interface{}, args []string, fn CommandFunc, descs
  ...string) int {
          desc := ""
          if len(descs) > 0 {
                  desc = strings.Join(descs, "\n")
  @@ -30,7 +30,9 @@ func RunWithArgs(argv interface{}, args []string, fn
  CommandFunc, descs ...strin
          }).Run(args[1:])
          if err != nil {
                  fmt.Fprintln(os.Stderr, err)
  +               return 1
          }
  +       return 0
   }

(Also fixes one typo in "run-examples.sh")

7 of 7 new or added lines in 3 files covered. (100.0%)

1419 of 2105 relevant lines covered (67.41%)

108.69 hits per line