Clarify chart name validation rules#2094
Conversation
Update the Chart Names section to explicitly document that: - Names must start and end with a lowercase letter or number - Leading/trailing dashes are invalid - helm lint validates these rules This aligns documentation with helm/helm#32075 which adds stricter chart name validation to the linter. Signed-off-by: promptless[bot] <promptless[bot]@users.noreply.github.com>
|
Just a reminder: If you'd like me to act on any feedback you have via Github comments, just type @Promptless in your suggestion and I'll get right on it! (I won't show up in the user dropdown, but I'll process any request that has @Promptless in the comment body.) |
| Invalid chart names include: | ||
| - Names with uppercase letters (e.g., `MyChart`) | ||
| - Names with underscores (e.g., `my_chart`) | ||
| - Names with dots (e.g., `my.chart`) |
There was a problem hiding this comment.
Names with dots (e.g.,
my.chart)
This is the only change in this PR that might be technically inaccurate. Previously, the docs said that you "should not" use dots in chart names.
I am not certain if the word "should" implies that dots are invalid for chart names, or if they are just discouraged.
@TerryHowe , would you be able to confirm one way or the other re: the use of dots in chart names? If we're fine with the claim that "chart names are invalid if they include dots" then I think this PR can be merged as is.
2 participants
Open this suggestion in Promptless to view citations and reasoning process
Update chart naming documentation to explicitly state that names must start and end with a lowercase letter or number, list invalid name patterns, and note that helm lint validates these rules.
Trigger Events
Tip: Use labels in the Promptless dashboard to categorize suggestions by release or team 🏷️