From 69feb96490a1368d1ad693712f054c90c6093791 Mon Sep 17 00:00:00 2001 From: Thomas O'Donnell Date: Fri, 6 Sep 2019 15:42:05 +0200 Subject: [PATCH] Cleanup nitpick suggestion in the lint command Have fixed up a non-blocking nitpick change from #6310, this just switches us to use a regular string rather than a string.Builder for the summary. Signed-off-by: Thomas O'Donnell --- cmd/helm/lint.go | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/cmd/helm/lint.go b/cmd/helm/lint.go index e124aa83e..8b18d3261 100644 --- a/cmd/helm/lint.go +++ b/cmd/helm/lint.go @@ -91,12 +91,11 @@ func newLintCmd(out io.Writer) *cobra.Command { fmt.Fprintf(out, message.String()) - var summary strings.Builder - fmt.Fprintf(&summary, "%d chart(s) linted, %d chart(s) failed", len(paths), failed) + summary := fmt.Sprintf("%d chart(s) linted, %d chart(s) failed", len(paths), failed) if failed > 0 { - return errors.New(summary.String()) + return errors.New(summary) } - fmt.Fprintf(out, "%s\n", summary.String()) + fmt.Fprintln(out, summary) return nil }, }