Open zeljkobekcic opened 7 months ago
Hi thanks for the submission of the feature request,
currently there is no support at all for editing values.yaml files, so your nvim config is kind of wrong for even having helm-ls enabled for those files.
I realize that completion inside values files is not supported yet, however it would be really helpful if completion of values of dependency charts would work inside templates.
So same as the example above, it would be nice if {{ .Values.nginx.
would show completion for the merged values of the nginx dependency chart.
It looks like from the design this is currently not supported? At least the Chart
in https://github.com/mrjosh/helm-ls/blob/master/internal/charts/chart.go does not yet have any reference to the actual dependency chart structs, correct?
What would be the best way of adding this? I could also try to contribute here.
Thanks for your support!
I realize that completion inside values files is not supported yet, however it would be really helpful if completion of values of dependency charts would work inside templates.
So same as the example above, it would be nice if
{{ .Values.nginx.
would show completion for the merged values of the nginx dependency chart.It looks like from the design this is currently not supported? At least the
Chart
in https://github.com/mrjosh/helm-ls/blob/master/internal/charts/chart.go does not yet have any reference to the actual dependency chart structs, correct?What would be the best way of adding this? I could also try to contribute here.
Thanks for your support!
You are right, the internal chart is missing the dependency information. Reinplementing this could become quit a lot of work, thats why I suggested https://github.com/mrjosh/helm-ls/pull/77. Either way it would probably be good to use the chart struct as implemented by helm (https://github.com/helm/helm/tree/main/pkg/chart or https://github.com/mrjosh/helm-ls/tree/master/pkg/chart) for this. Please also note that im currently refactoring completion etc. in https://github.com/mrjosh/helm-ls/pull/72
Ah thanks for the hint, that makes sense. Then this could definitely be tackled in the refactoring with #77 .
Then I guess it makes sense to wait until #72 is completed. Thanks for the insights.
I realize that completion inside values files is not supported yet, however it would be really helpful if completion of values of dependency charts would work inside templates.
So same as the example above, it would be nice if
{{ .Values.nginx.
would show completion for the merged values of the nginx dependency chart.It looks like from the design this is currently not supported? At least the
Chart
in https://github.com/mrjosh/helm-ls/blob/master/internal/charts/chart.go does not yet have any reference to the actual dependency chart structs, correct?What would be the best way of adding this? I could also try to contribute here.
Thanks for your support!
Hi @msvechla, feel free to test https://github.com/mrjosh/helm-ls/pull/80, which should do what you suggested and provide some feedback.
Hey,
I was trying to use the LS in NeoVim but the completion based on dependencies (in the Helm Chart) do not work and also going to the definition. Going to the definition of a value in the dependency would be nice.
A minimal case can be:
helm create github-issue-chart
Then copy over this
Chart.yaml
I would expect this Helm LS to be able to suggest values for
nginx.SOMETHING
inside thevalues.yaml
. This does not work either after running helm dependency update and even after un-tar-ing the tarball (chart).Here are the logs from
:LspLog
I am also using the latest version of
helm-ls
.Thank you in advance!