feat: add dprint lsp #2982
feat: add dprint lsp #2982
Conversation
|
the dprint lsp supports all filetypes that dprint supports. But these are all determined via configuration. So it's up to the user to decide what filetypes to use dprint for. So I left the filetypes empty. |
| return { | ||
| default_config = { | ||
| cmd = { 'dprint', 'lsp' }, | ||
| filetypes = {}, |
There was a problem hiding this comment.
It should contain file types that dprint officially supports: https://github.com/nvimtools/none-ls.nvim/blob/a311c7cc8f17543143a7482cdbe3a384c371d56a/lua/null-ls/builtins/formatting/dprint.lua#L17-L29
There was a problem hiding this comment.
I don't think null-ls should be used as a source of truth. dprint, when installed, doesn't come with any support for anything. You can later configure it for one or many different languages.
If placing all these filetypes in the filetypes is what it takes to add this to nvim-lspconfig I'll do it. I don't think having the filetypes be too expansive is that bad. And that's what I have now done.
WhoIsSethDaniel
force-pushed
the
add-dprint-lsp
branch
from
c36b5c8
to
44295ed
Compare
No description provided.