Missing value at path rule should be improved

tim-koehler / Helm-Intellisense

Extension for Visual Studio Code - Intellisense in helm-templates from the values.yaml

https://marketplace.visualstudio.com/items?itemName=Tim-Koehler.helm-intellisense&ssr=false#overview

MIT License

1.67k stars 25 forks source link

Missing value at path rule should be improved #27

Closed xrow closed 3 years ago

xrow commented 3 years ago

First off... thanks for this idea/tool. I am currently testing, if it is helpfull. I noticed the following when usign it with my charts:

In this example you see the the tool is generating an error. This is indeed correct that .Values.externalServices.mysql is not existing, but at the same time it is my desire to have the default empty. That is the reason why im am testing for the value.

Futhermore I do not know what to recommend. I just wanted to let you know.

tim-koehler commented 3 years ago

Hey @xrow, thanks for informing me about this. I adjusted the regex for the Linter to not only ignore the {{ if... but also the {{ else if... lines. I plan on introducing an option in the settings to enable/disable linting for those if/else conditions. For now the Linter will just ignore these.

tim-koehler commented 3 years ago

Fix released with v0.9.1