feat: add delphilsp support #3034
Conversation
| if dpr_path then | ||
| return vim.fs.find(function(name) | ||
| return name:match '.*%.delphilsp.json$' | ||
| end, { type = 'file', path = vim.fs.dirname(dpr_path), upward = false })[1] |
There was a problem hiding this comment.
Unfortunately, it is required. The dpr_path contains file name that needs to be removed for vim.fs.find to work.
It would be nice to replace the dpr_path with root_dir, as Delphi's lsp config and project file are always located in the root folder. With current implementation we have to look for the same folder twice, first time for lsp config and second time for the root_dir
There was a problem hiding this comment.
what about if find return an empty table ?
There was a problem hiding this comment.
Setting DelphiLSP configuration on server setup was wrong, @glepnir thank you for being patient with me.
The proper way is to send it to LS through didChangeConfiguration on on_attach.
At this point the root_dir is already set.
I've added a configuration sample in the documentation section of delphi_ls.
Add support for Delphi Language Server, which is available only for Windows.
The config file for the language server *.delphilsp.json is located in the root_dir.
Unfortunately the root_dir is not available at the configuration creation time,
so I had to look for it. Maybe, there is a better solution?