temporalio / helm-charts

Temporal Helm charts
MIT License
294 stars 321 forks source link

fix: disable whitespace removal for include temporal.serviceAccount #381

Closed 2hamed closed 1 year ago

2hamed commented 1 year ago

Fixes issue: https://github.com/temporalio/helm-charts/issues/380

What was changed

Removed the hyphen (-) at the beginning of the include function. It was causing all whitespaces to be removed from the output resulting in serviceAccount: ... to be on the same line as the previous statement.

CLAassistant commented 1 year ago

CLA assistant check
All committers have signed the CLA.

CLAassistant commented 1 year ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

crhuber commented 1 year ago

@rodrigozhou afaict the lastest release is broken until this is merged