-
Notifications
You must be signed in to change notification settings - Fork 3
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
LSP Issues #7
Comments
Hi, you need to install
|
Does ocamllsp support
Sorry if this is off topic, I'm pretty new to the OCaml ecosystem but this project looks very cool. Thanks! |
@jakequinter do you use https://github.com/ocaml-mlx/ocaml_mlx.nvim plugin? |
another thing to check is the version of ocaml-lsp-server package, only the recent version support .mlx |
so i was indeed on an older version of the ocaml compiler which resulted in an older version of ocaml-lsp-server. i've updated that to the latest which appears to be I am using ocaml_mlx.nvim like this: https://github.com/jakequinter/configuration/blob/main/.config/nvim/lua/jdq/plugins/ocaml-mlx.lua not sure if something in my lsp config is borked or something. I definitely don't expect you to look into it...I'm going to keep digging, but my ocaml lsp config does includes |
although i think the problem may be related to it using mason. may need to look at trying to get it to default to the cwd |
figured it out. just incase anyone is viewing this in the future, i needed to tell vim that the filetype of "ocaml_mlx" should return "ocaml" something like:
thanks for the help andreypopp and a huge thanks for the great work! |
Hey! Thanks for the awesome work :)
One reason I'm interested in this project is because LSP support for ReasonML is not working as expected on react components. So I tried cloning this template and fired up my lsp, which is Emacs' Eglot (using
ocamllsp
). And I encountered the following error immediately on startup:Any idea why this is going on? I saw this announcement of the project which mentioned the LSP functioning, but the template above does include the dune expression setting the dialect.
Thanks a ton!
The text was updated successfully, but these errors were encountered: