cetic / helm-zabbix

Helm Chart For Zabbix
https://artifacthub.io/packages/helm/cetic/zabbix
Apache License 2.0
56 stars 57 forks source link

Fix missing newline that results in invalid YAML #56

Closed acritelli closed 2 years ago

acritelli commented 2 years ago

Signed-off-by: Anthony Critelli anthony@cloudify.co

What this PR does / why we need it:

This PR addresses a missing newline that causes chart rendering issues. See #55

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

Special notes for your reviewer:

Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

aeciopires commented 2 years ago

Hi @acritelli!

Thanks for your contributition and sorry for this problem.

@sa-ChristianAnton you agree with this solution? Can I publish this hotfix before 3.0.0 version?

@acritelli for your information, we are working in 3.0.0 version of chart in this PR: https://github.com/cetic/helm-zabbix/pull/54