From 9bb208ddf1fc9788f3ed3780ac61a90d1678eeac Mon Sep 17 00:00:00 2001 From: Gabriel Guzman Date: Mon, 30 Oct 2017 14:06:53 -0400 Subject: [PATCH] Minor lint cleanups Wasn't sure about the unused variable: `var defaultIssue = &JiraIssue{}` let me know if that needs to stay there for some reason. I built and installed `jkl` and quick tests show things appear to still be working. --- cmd/jkl/jkl.go | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/cmd/jkl/jkl.go b/cmd/jkl/jkl.go index 45da524..daa8016 100644 --- a/cmd/jkl/jkl.go +++ b/cmd/jkl/jkl.go @@ -55,7 +55,7 @@ func getCmd(args []string, depth int) (Runner, error) { case "edit": return NewEditCmd(args[1:]) case "comment": - if strings.Contains(strings.Join(args,""),jkl.CommentIdSeparator){ + if strings.Contains(strings.Join(args, ""), jkl.CommentIdSeparator) { return NewEditCommentCmd(args[1:]) } return NewCommentCmd(args[1:]) @@ -80,21 +80,21 @@ func getCmd(args []string, depth int) (Runner, error) { } args[0], args[1] = args[1], args[0] return getCmd(args, depth+1) - } else { - // Swapping the first two args didn't help; - // this means it's a transition. - - // tcmd, err := NewTransitionCommand(args) - // if err != nil {return nil, err} - // return tcmd, nil } + // Swapping the first two args didn't help; + // this means it's a transition. + + // tcmd, err := NewTransitionCommand(args) + // if err != nil {return nil, err} + // return tcmd, nil } return nil, ErrTaskSubCommandNotFound } -var verbs = []string{"list", "create", "task", "edit", "comment","edit-comment"} -func init(){ -sort.Strings(verbs) +var verbs = []string{"list", "create", "task", "edit", "comment", "edit-comment"} + +func init() { + sort.Strings(verbs) } const usage = `Usage: