WARNING: THIS SITE IS A MIRROR OF GITHUB.COM / IT CANNOT LOGIN OR REGISTER ACCOUNTS / THE CONTENTS ARE PROVIDED AS-IS / THIS SITE ASSUMES NO RESPONSIBILITY FOR ANY DISPLAYED CONTENT OR LINKS / IF YOU FOUND SOMETHING MAY NOT GOOD FOR EVERYONE, CONTACT ADMIN AT ilovescratch@foxmail.com
Skip to content

Conversation

@bfritz
Copy link

@bfritz bfritz commented Dec 7, 2025

Removed tabs in Chart.yaml that were causing helm lint and helm package to fail, including in CI. Also added .editorconfig file to help with yaml forwarding moving forward.

Finally, exclude a few more files in the repo from the published helm chart.

@bfritz
Copy link
Author

bfritz commented Dec 7, 2025

Offering this up so helm package will work. During a quick search, I didn't see a contributing guide, so I made some guesses around changelog format, versioning, etc. Happy to adjust the PR if you'd like anything handled differently.

Removed tabs in `Chart.yaml` that were causing `helm lint` and
`helm package` to fail, including in CI.  Also added `.editorconfig`
file to help with yaml forwarding moving forward.

Finally, exclude a few more files in the repo from the published
helm chart.
@bfritz bfritz force-pushed the fix/chart_yaml_errors branch from d7404f3 to b5b9689 Compare December 7, 2025 16:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant