nvim-lspconfig
nvim-lspconfig copied to clipboard
feat: add dagger CUE lsp
https://github.com/mattn/vim-lsp-settings mentioned in https://github.com/dagger/daggerlsp/pull/72#issue-1314001216 is related for those that are still on vim. I use vim on my legacy workstation (Mac 2017) & nvim on my 2 newer workstations (Mac 2021 & Linux 2022).
@sigmaSd apologies beforehand for the direct ping. Any chances to quickly review this? Should be straightforward to merge :pray:
@marcosnils well I'm not a maintainer of this repo xD good luck though!
whoop! my bad. @justinmk maybe can help here? We're about to announce the release our LSP this week and it'd be great to have this merged.
I didn't directly use this patch — but was inspired by it.
I got it to work with the following configuration.
local lspconfig = require 'lspconfig'
local configs = require 'lspconfig.configs'
local util = require 'lspconfig.util'
if not configs.dagger then
configs.dagger = {
default_config = {
cmd = { 'dagger', 'cuelsp' },
filetypes = { 'cue' },
root_dir = function(fname)
return util.root_pattern( 'cue.mod', '.git' )( fname )
end,
single_file_support = true,
settings = {}
},
}
end
lspconfig.dagger.setup { }
@lithammer @justinmk is there a way we can help to get this merged? I see other newer PR's get approved / merged and so far we haven't received any inputs on this one :pray:
There's some formatting (Stylua) issues.
There's some formatting (Stylua) issues.
that's unrelated to our changes. GA is complaining due to some dartls configs which have not been modified in this PR. Maybe a previous PR has been merged with failing CI?
There's some formatting (Stylua) issues.
that's unrelated to our changes. GA is complaining due to some
dartlsconfigs which have not been modified in this PR. Maybe a previous PR has been merged with failing CI?
Right, sorry. I clearly wasn't looking very carefully 😓
Right, sorry. I clearly wasn't looking very carefully
no worries. Thx for merging :hugs: