can't publish charts with semver build postfix (aka, cr breaks if + in chart version)

[itewk]

Issue

helm states that the version in Chart.yaml supports SemVer, which includes an option for a build section (+BUILD) (https://semver.org/#backusnaur-form-grammar-for-valid-semver-versions).

But, when i try to use the chart releaser with a helm chart with a version of 0.1.0-edge1+SHA-abc123 it fails, if I drop the + to 0.1.0-edge1 it works. based on the error message it looks like some issues with escaping the +.

Logs

Run helm/chart-releaser-action@v1.0.0
tssc-tekton-cloud-resources
Looking up latest tag...
Discovering changed charts since 'ceed80e8ab27acbabccbbd21434b43ee19d0abfb'...
Installing chart-releaser...
Packaging chart 'charts/tssc-workflow-tekton-cluster-resources'...
Successfully packaged chart and saved it to: .cr-release-packages/tssc-workflow-tekton-cluster-resources-0.1.0-edge8+shaceed80e8.tgz
Packaging chart 'charts/tssc-workflow-tekton-pipeline'...
walk.go:74: found symbolic link in path: /home/runner/work/tssc-tekton-cloud-resources/tssc-tekton-cloud-resources/charts/tssc-workflow-tekton-pipeline/charts/tssc-workflow-tekton-shared-resources resolves to /home/runner/work/tssc-tekton-cloud-resources/tssc-tekton-cloud-resources/charts/tssc-workflow-tekton-shared-resources
walk.go:74: found symbolic link in path: /home/runner/work/tssc-tekton-cloud-resources/tssc-tekton-cloud-resources/charts/tssc-workflow-tekton-pipeline/charts/tssc-workflow-tekton-shared-resources resolves to /home/runner/work/tssc-tekton-cloud-resources/tssc-tekton-cloud-resources/charts/tssc-workflow-tekton-shared-resources
Successfully packaged chart and saved it to: .cr-release-packages/tssc-workflow-tekton-pipeline-0.1.0-edge8+shaceed80e8.tgz
Packaging chart 'charts/tssc-workflow-tekton-shared-resources'...
Successfully packaged chart and saved it to: .cr-release-packages/tssc-workflow-tekton-shared-resources-0.1.0-edge8+shaceed80e8.tgz
Releasing charts...
+ cr index -o itewk -r tssc-tekton-cloud-resources -c https://itewk.github.io/tssc-tekton-cloud-resources
Updating charts repo index...
====> UpdateIndexFile new index at .cr-index/index.yaml
Error: .cr-release-packages/tssc-workflow-tekton-cluster-resources-0.1.0-edge8%2Bshaceed80e8.tgz is not a helm chart package: stat .cr-release-packages/tssc-workflow-tekton-cluster-resources-0.1.0-edge8%2Bshaceed80e8.tgz: no such file or directory
====> Found tssc-workflow-tekton-cluster-resources-0.1.0-edge8+shaceed80e8.tgz
Usage:
  cr index [flags]

[jlpedrosa]

Same here, actually the hot potatoe is in the mix between GH which provides a link url escaped (https://github.com/jlpedrosa/helm-chart/releases/download/2.11.2%2BChart1/unpoller-2.11.2+Chart1.tgz) and the chart releaser, not the action.

helm/chart-releaser#406 this fixes it.

[karlyan]

same here. hope to add this in new release. so we can use https://semver.org/#spec-item-10