Formatting via yamlls

[franco-ruggeri]

As mentioned in #32 (at the end of OP), formatting via yamlls is not supported yet, but would be a nice feature to add.

Opening this issue as a placeholder.

[qvalentin]

Yes, this is still interesting, but it probably won't be as easy as I thought it would be back then.

See also golang/go#69413 and prettier/prettier#6517

[digitalstudium]

Just wrote formatter for helm templates: https://github.com/digitalstudium/helmfmt

[eshepelyuk]

I assume it's hard to implement yamlls formatting for template files, but what about values.yaml ?

They are plain YAML so delegating the formatting to yamlls should be sufficient, am I missing smth ?

[franco-ruggeri]

For me, values.yaml gets file type yaml.helm-values, and yamlls attaches to the buffer correctly in addition to, and separately from, helm_ls. So, formatting works fine in values.yaml.

[eshepelyuk]

What do you see when invoking ?

:lua vim.lsp.buf.format()

[qvalentin]

I assume it's hard to implement yamlls formatting for template files, but what about values.yaml ?

They are plain YAML so delegating the formatting to yamlls should be sufficient, am I missing smth ?

I would recommend just running yaml-language-server next to helm-ls for values.yaml as @franco-ruggeri said.

You might need to configure yaml.format.enable: https://github.com/redhat-developer/yaml-language-server?tab=readme-ov-file#language-server-settings

Alternatively you could also simply use none-ls with prettier.

[eshepelyuk]

I would recommend just running yaml-language-server next to helm-ls for values.yaml as @franco-ruggeri said.

I have both LSP servers attached to the buffer, i see it by running :LspInfo, but when i run format via :lua vim.lsp.buf.format() it says LSP] Format request failed, no matching language servers.

Also I've tried this lua vim.lsp.buf.format({name="yamlls"}) - receiving the same message.

I'm assuming it's related to file type yaml.helm-values, if i set file type to yaml - yamlls formatting works, even with helm_ls client attached.

Maybe I should raise this in NeoVim plugin repo instead ?