From e7445a463298f65a5468e65e0b910ffb96479755 Mon Sep 17 00:00:00 2001 From: Matt Butcher Date: Fri, 5 Feb 2016 14:13:05 -0700 Subject: [PATCH] fix(vet): fix a number of lint/vet warnings --- cmd/helm/dm.go | 2 +- cmd/helm/doctor.go | 1 + dm/uninstall.go | 2 +- format/messages.go | 2 ++ 4 files changed, 5 insertions(+), 2 deletions(-) diff --git a/cmd/helm/dm.go b/cmd/helm/dm.go index c5069900a..b331e62c7 100644 --- a/cmd/helm/dm.go +++ b/cmd/helm/dm.go @@ -27,7 +27,7 @@ func uninstall(dryRun bool) error { out, err := dm.Uninstall(runner) if err != nil { - format.Error("Error uninstalling: %s %s", out, err) + format.Err("Error uninstalling: %s %s", out, err) } format.Msg(out) return nil diff --git a/cmd/helm/doctor.go b/cmd/helm/doctor.go index ebf66099a..15f068a25 100644 --- a/cmd/helm/doctor.go +++ b/cmd/helm/doctor.go @@ -2,6 +2,7 @@ package main import ( "github.com/deis/helm-dm/dm" + "github.com/deis/helm-dm/format" "github.com/deis/helm-dm/kubectl" ) diff --git a/dm/uninstall.go b/dm/uninstall.go index 277ff1c10..675ebdb73 100644 --- a/dm/uninstall.go +++ b/dm/uninstall.go @@ -4,7 +4,7 @@ import ( "github.com/deis/helm-dm/kubectl" ) -// uninstall uses kubectl to uninstall the base DM. +// Uninstall uses kubectl to uninstall the base DM. // // Returns the string output received from the operation, and an error if the // command failed. diff --git a/format/messages.go b/format/messages.go index 59de3e6cb..b1c63d510 100644 --- a/format/messages.go +++ b/format/messages.go @@ -24,11 +24,13 @@ func Msg(msg string, v ...interface{}) { fmt.Fprintf(os.Stdout, msg, v...) } +// Success is an achievement marked by pretty output. func Success(msg string, v ...interface{}) { msg = "[Success] " + msg + "\n" fmt.Fprintf(os.Stdout, msg, v...) } +// Warning emits a warning message. func Warning(msg string, v ...interface{}) { msg = "[Warning] " + msg + "\n" fmt.Fprintf(os.Stdout, msg, v...)