Merge pull request #118 from overbool/fix/timeout-error

Stebalien · web-flow · commit 6aaaa76dfeba · 2018-09-12T21:34:01.000Z
fix(option): print error message for error timeout option
diff --git a/cli/run.go b/cli/run.go
@@ -39,6 +39,7 @@ func Run(ctx context.Context, root *cmds.Command,
 	if timeoutStr, ok := req.Options[cmds.TimeoutOpt]; ok {
 		timeout, err := time.ParseDuration(timeoutStr.(string))
 		if err != nil {
+			printErr(err)
 			return err
 		}
 		req.Context, cancel = context.WithTimeout(req.Context, timeout)

Original file line number	Diff line number	Diff line change
`@@ -39,6 +39,7 @@ func Run(ctx context.Context, root *cmds.Command,`
`39`	`39`	`if timeoutStr, ok := req.Options[cmds.TimeoutOpt]; ok {`
`40`	`40`	`timeout, err := time.ParseDuration(timeoutStr.(string))`
`41`	`41`	`if err != nil {`
	`42`	`+ printErr(err)`
`42`	`43`	`return err`
`43`	`44`	`}`
`44`	`45`	`req.Context, cancel = context.WithTimeout(req.Context, timeout)`