-
Notifications
You must be signed in to change notification settings - Fork 192
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
[Enhancement] Rename Requirements to Dependencies #88
Comments
Agreed, I'll merge a PR, or get to this myself... someday soon |
Hi @norwoodj , I'm using your project for our Helm charts documentation - thanks for that - and I was looking for a Cheers! |
Just bumped into this issue after trying to figure out why the Think the mentioned pull request #161 looks like a solution to me. Anything that would have to be tweaked on the PR to make it increase it's chances of being merged? |
Following up on my own message to avoid that anyone else gets trapped in the same way: My issue was caused by a wrong value in my The mentioned PR and this issue are about consistent naming only I think. |
I would be nice to have the nomenclature of the requirements/dependencies match helm to avoid confusion. Helm documents them as dependencies whereas helm-docs state them as requirements.
The text was updated successfully, but these errors were encountered: