helm-docs icon indicating copy to clipboard operation
helm-docs copied to clipboard

Add an optional flag to remove the helm-docs version footer from the default template

Open gbprz opened this issue 1 year ago • 3 comments

This change adds a new flag named skip-version-footer. When set to true, the version footer is removed from the default README template. By default, the flag is set to false so the version footer is kept for backwards compatibility.

I had a similar issue as this comment https://github.com/norwoodj/helm-docs/issues/162#issuecomment-1819415642 where the default README template is great for the majority of our charts. For maintenance, it would be much easier for us to avoid copying README templates for each chart just to remove the version footer.

Fixes https://github.com/norwoodj/helm-docs/issues/162

gbprz avatar Feb 27 '24 17:02 gbprz

I guess this PR would supercede my one to update at least the documentation https://github.com/norwoodj/helm-docs/pull/219

mpluhar avatar Feb 27 '24 17:02 mpluhar

There have been requests to do this in the past and I still feel the same way:

If you don't want the versionFooter in your template, you can use a README.md.gotmpl file to define the template that you want to use when rendering your README. The default template is what it is, if you don't like it, there is already a mechanism to define your own. I don't want to go down the road of adding a ton of options to augment the default template like this.

norwoodj avatar Mar 02 '24 14:03 norwoodj

There have been requests to do this in the past and I still feel the same way:

If you don't want the versionFooter in your template, you can use a README.md.gotmpl file to define the template that you want to use when rendering your README. The default template is what it is, if you don't like it, there is already a mechanism to define your own. I don't want to go down the road of adding a ton of options to augment the default template like this.

Could you comment on the idea of keeping the footer but removing the version? I think that would solve the problem of unexpected/untimely changes that complicated workflows such as validation in CD and pre-commit.

Thanks!

hstenzel avatar Apr 02 '24 18:04 hstenzel

I'll merge it. Thanks for the changes, I've been convinced.

norwoodj avatar Jul 06 '24 14:07 norwoodj