Skip to content

Commit 2d67ec2

Browse files
committed
Mage: Stop mutating release notes.
1 parent dbe4994 commit 2d67ec2

File tree

1 file changed

+13
-12
lines changed

1 file changed

+13
-12
lines changed

magefiles/steps/helm.go

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,7 @@ package steps
1818

1919
import (
2020
"bytes"
21-
"fmt"
2221
"os"
23-
"strings"
2422

2523
semver "github.com/blang/semver/v4"
2624
"github.com/helm/helm/pkg/chartutil"
@@ -104,18 +102,21 @@ func updateVersion(version string) {
104102
utils.CheckIfError(err, "HELM Saving new Chart")
105103
}
106104

107-
func updateChartReleaseNotes(releasesNotes []string) {
108-
utils.Info("HELM Updating the Chart Release notes")
105+
func updateChartReleaseNotes(releaseNotes []string) {
106+
utils.Info("HELM Updating chart release notes")
109107
chart, err := chartutil.LoadChartfile(HelmChartPath)
110-
utils.CheckIfError(err, "HELM Could not Load Chart to update release notes %s", HelmChartPath)
111-
for i := range releasesNotes {
112-
releasesNotes[i] = fmt.Sprintf("- %q", releasesNotes[i])
113-
}
114-
releaseNoteString := strings.Join(releasesNotes, "\n")
115-
utils.Info("HELM Release note string %s", releaseNoteString)
116-
chart.Annotations["artifacthub.io/changes"] = releaseNoteString
108+
utils.CheckIfError(err, "HELM Failed to load chart manifest: %s", HelmChartPath)
109+
110+
releaseNotesBytes, err := yaml.Marshal(releaseNotes)
111+
utils.CheckIfError(err, "HELM Failed to marshal release notes")
112+
113+
releaseNotesString := string(releaseNotesBytes)
114+
utils.Info("HELM Chart release notes:\n%s", releaseNotesString)
115+
chart.Annotations["artifacthub.io/changes"] = releaseNotesString
116+
117+
utils.Info("HELM Saving chart release notes")
117118
err = chartutil.SaveChartfile(HelmChartPath, chart)
118-
utils.CheckIfError(err, "HELM Saving updated release notes for Chart")
119+
utils.CheckIfError(err, "HELM Failed to save chart manifest: %s", HelmChartPath)
119120
}
120121

121122
// UpdateChartValue Updates the Helm ChartValue

0 commit comments

Comments
 (0)