fix: removing "trim" function - Githubissues

Joxit / helm-charts

Official Helm Chart Repository for Joxit Applications

https://helm.joxit.dev/

MIT License

12 stars 7 forks source link

fix: removing "trim" function #12

Closed nurimansyah89 closed 1 year ago

nurimansyah89 commented 1 year ago

Issue

When using Helm V3, if we set array on key that convert with toYaml function, it's produce mapping value not allowed on context error

Reproduce

Using from values.yaml

Change, for example, tolerations with some values:

...
tolerations:
- key: "some-key"
  operator: "Exists"
  effect: "NoSchedule"
...

Run helm install command
Error triggered

Actual

Mapping values are not allowed on context error showed

Expected

No error on run

Additional Proof

This will fix #11

Joxit commented 1 year ago

Hi, thank you for using my project and contribution to it :smile:

Nice catch and LGTM !