search

vidispine / hull

The incredible HULL - Helm Uniform Layer Library - is a Helm library chart to improve Helm chart based workflows
https://github.com/vidispine/hull
Apache License 2.0
231 stars 13 forks source link

appVersion type error in Chart.yaml #169

Closed matthias4217 closed 2 years ago

matthias4217 commented 2 years ago

Helm lint (Helm version 3.10.0, 3.8.2, 3.5.0) gives the following :

$ helm lint .
==> Linting .
[ERROR] Chart.yaml: appVersion should be of type string but it's of type float64
[INFO] Chart.yaml: icon is recommended

Error: 1 chart(s) linted, 1 chart(s) failed

Indeed in Chart.yaml, we've got appVersion: 1.25. The value should be quoted, as it is currently interpreted as a float.

This is quite a minor detail, but it may break workflows relying on linting to check the validity of the chart.

gre9ory commented 2 years ago

Thanks for that! I intend to repair it and close the issue with the next set of releases coming soon!