Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

.gitattributes: Mark install/kubernetes/cilium/README.md as generated #24295

Merged
merged 1 commit into from Mar 13, 2023

Conversation

qmonnet
Copy link
Member

@qmonnet qmonnet commented Mar 10, 2023

The README.md file inside of the install/kubernetes/cilium/ directory has been generated with helm-docs ever since its introduction in commit 7295879, using README.md.gotmpl as a template, and other files such as values.yaml (in turn generated from values.yaml.tmpl). It should not be edited by hand. Pull Requests on GitHub usually display changes on this README.md, but instead we want to focus on the source files from which the changes are generated. Let's mark the file as generated, so that the GitHub interface hides the related diff, and reviewers don't lose time looking at it.

@qmonnet qmonnet added the release-note/misc This PR makes changes that have no direct user impact. label Mar 10, 2023
@qmonnet qmonnet requested a review from a team as a code owner March 10, 2023 13:28
The README.md file inside of the install/kubernetes/cilium/ directory
has been generated with helm-docs ever since its introduction in commit
7295879 ("build(helm): Add makefile target to generate helm
README.md"), using README.md.gotmpl as a template, and other files such
as values.yaml (in turn generated from values.yaml.tmpl). It should not
be edited by hand. Pull Requests on GitHub usually display changes on
this README.md, but instead we want to focus on the source files from
which the changes are generated. Let's mark the file as generated, so
that the GitHub interface hides the related diff, and reviewers don't
lose time looking at it.

Signed-off-by: Quentin Monnet <quentin@isovalent.com>
@qmonnet
Copy link
Member Author

qmonnet commented Mar 13, 2023

This file is not used at all by the CI, it only helps GitHub interface rendering diffs, so we're good to go.

@qmonnet qmonnet added the ready-to-merge This PR has passed all tests and received consensus from code owners to merge. label Mar 13, 2023
@borkmann borkmann merged commit 70c2da1 into cilium:master Mar 13, 2023
@qmonnet qmonnet deleted the pr/gitattributes-readme branch March 14, 2023 09:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready-to-merge This PR has passed all tests and received consensus from code owners to merge. release-note/misc This PR makes changes that have no direct user impact.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants