fix: allow multiple cronjobs

stakater / application

Generic helm chart for all kind of applications

http://stakater.com

MIT License

202 stars 85 forks source link

fix: allow multiple cronjobs #271

Closed aslafy-z closed 11 months ago

aslafy-z commented 12 months ago

The cronjob template misses a --- new document separation. This means all cronjobs are scrambled together and only a frankenstein version of all of them is emitted. This PR also does some lint:

move the new document separator to the beginning of templates
add empty newline at the end of files
remove empty newlines in the middle of files
move helpers to _helpers file.

github-actions[bot] commented 11 months ago

@mustafaStakater validation successful`