-
Notifications
You must be signed in to change notification settings - Fork 3
Open
Description
Hi,
Thanks for a great plugin!
I couldn’t use it at the start because the function get_server looks for a client named ocamllsp, and in my config I called mine ocaml_ocamllsp. I suggest to have a config option to be able to set this name to something else.
This was fixed for me with this commit, but it’s only replacing the hardcoded string by another one.
Another thing: when it failed, it did print “Cannot find LSP server”, but then did try to use the returned result at this line, causing a Lua error and stack trace. It should probably stop after printing the error—though, in this case, the stack trace did help me find the problem quickly.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels