Merge pull request #432 from michelleN/helm-repo

ref(repo:) use run function for error handling
pull/436/head
Michelle Noorali 9 years ago
commit 2c295a4ca5

@ -20,7 +20,6 @@ import (
"errors" "errors"
"github.com/codegangsta/cli" "github.com/codegangsta/cli"
"github.com/kubernetes/helm/pkg/format" "github.com/kubernetes/helm/pkg/format"
"os"
) )
func init() { func init() {
@ -45,12 +44,7 @@ func repoCommands() cli.Command {
Usage: "The name of the credential.", Usage: "The name of the credential.",
}, },
}, },
Action: func(c *cli.Context) { Action: func(c *cli.Context) { run(c, addRepo) },
if err := addRepo(c); err != nil {
format.Err("%s Error adding chart repo: ", err)
os.Exit(1)
}
},
}, },
{ {
Name: "show", Name: "show",
@ -61,24 +55,14 @@ func repoCommands() cli.Command {
Name: "list", Name: "list",
Usage: "List the repositories on the remote manager.", Usage: "List the repositories on the remote manager.",
ArgsUsage: "", ArgsUsage: "",
Action: func(c *cli.Context) { Action: func(c *cli.Context) { run(c, listRepos) },
if err := listRepos(c); err != nil {
format.Err("%s Error listing chart repositories: ", err)
os.Exit(1)
}
},
}, },
{ {
Name: "remove", Name: "remove",
Aliases: []string{"rm"}, Aliases: []string{"rm"},
Usage: "Remove a repository from the remote manager.", Usage: "Remove a repository from the remote manager.",
ArgsUsage: "REPOSITORY", ArgsUsage: "REPOSITORY",
Action: func(c *cli.Context) { Action: func(c *cli.Context) { run(c, removeRepo) },
if err := removeRepo(c); err != nil {
format.Err("%s Error removing chart repo: ", err)
os.Exit(1)
}
},
}, },
}, },
} }

Loading…
Cancel
Save