HTML: conform "formatters_by_ft" not set up if lsp is enabled

[vnphanquang]

Hello there, thank you for the good work! I noticed here:

nvf/modules/plugins/languages/html.nix

Lines 117 to 123 in b05fade

	(mkIf (cfg.format.enable && !cfg.lsp.enable) {
	vim.formatter.conform-nvim = {
	enable = true;
	presets = genAttrs cfg.format.type (_: {enable = true;});
	setupOpts.formatters_by_ft.html = cfg.format.type;
	};
	})

...i.e, vim.languages.html.lsp.enable is true, formatter.conform-nvim.setupOpts.formatters_by_ft.html is skipped.

Is there a reason behind this? It threw me off since there isn't any note about this in the docs and I had expected conform to pick up the right formatter in HTMl files automatically, which wasn't the case.

As a workaround I did specify formatters_by_ft.html manually. Just wondering if there is any gotcha i'm not aware of.

[horriblename]

That was to prevent the default LSP and formatters, which are both superhtml, to be added at the same time. That said, the condition is a bit too loose, will update it

[vnphanquang]

Thank you.