Skip to content

feat(values): lint unused values in values.yaml#177

Draft
qvalentin wants to merge 3 commits intomasterfrom
feat/lint-unused-values
Draft

feat(values): lint unused values in values.yaml#177
qvalentin wants to merge 3 commits intomasterfrom
feat/lint-unused-values

Conversation

@qvalentin
Copy link
Copy Markdown
Collaborator

@qvalentin qvalentin commented Nov 1, 2025

First draft for #176

Currently only runs correctly when you safe the file after it was opened.

image
  • Make it configurable
  • What about subcharts?
  • Tests
  • Run it after all templates were loaded

@calamity-m
Copy link
Copy Markdown

Random person coming along, but is there anything we can do to have this merged? I think having the diagnostic is pretty helpful.

I've just found helm-ls and being able to do this would be pretty cool, as a lot of issues I end up seeing with helm is people typo'ing or using incorrect indentation and not noticing until way later in their values.yaml or overrides.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants