From 4735f2be9b7669e4f7a1bdf4a6dc4522c4bdfe64 Mon Sep 17 00:00:00 2001 From: myeunee Date: Mon, 9 Sep 2024 02:19:50 +0900 Subject: [PATCH 1/2] Update history.go Rename formatChartname to formatChartName for consistency Signed-off-by: myeunee --- cmd/helm/history.go | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/cmd/helm/history.go b/cmd/helm/history.go index 7edb4767c..56383bfca 100644 --- a/cmd/helm/history.go +++ b/cmd/helm/history.go @@ -136,7 +136,7 @@ func getHistory(client *action.History, name string) (releaseHistory, error) { func getReleaseHistory(rls []*release.Release) (history releaseHistory) { for i := len(rls) - 1; i >= 0; i-- { r := rls[i] - c := formatChartname(r.Chart) + c := formatChartName(r.Chart) s := r.Info.Status.String() v := r.Version d := r.Info.Description @@ -159,7 +159,7 @@ func getReleaseHistory(rls []*release.Release) (history releaseHistory) { return history } -func formatChartname(c *chart.Chart) string { +func formatChartName(c *chart.Chart) string { if c == nil || c.Metadata == nil { // This is an edge case that has happened in prod, though we don't // know how: https://github.com/helm/helm/issues/1347 From de18ac16027201bb201c4cfe8736202d9feda8cd Mon Sep 17 00:00:00 2001 From: Terry Howe Date: Tue, 24 Sep 2024 08:33:07 -0600 Subject: [PATCH 2/2] fix: add missing formatChartName call Signed-off-by: Terry Howe --- cmd/helm/list.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/cmd/helm/list.go b/cmd/helm/list.go index 7e828c2b2..bbc7e979d 100644 --- a/cmd/helm/list.go +++ b/cmd/helm/list.go @@ -156,7 +156,7 @@ func newReleaseListWriter(releases []*release.Release, timeFormat string, noHead Namespace: r.Namespace, Revision: strconv.Itoa(r.Version), Status: r.Info.Status.String(), - Chart: formatChartname(r.Chart), + Chart: formatChartName(r.Chart), AppVersion: formatAppVersion(r.Chart), }