Ulysse comments

Results 310 comments of


                                            Ulysse

ocamllsp/switchImplIntf results in "Jsonrpc: json conversion failed: missing field error"

Indeed, it looks like the query expects an (json) array with a single uri: https://github.com/ocaml/ocaml-lsp/blob/master/ocaml-lsp-server/src/custom_requests/req_switch_impl_intf.ml#L13-L14 This is explained by the base protocol LSP relies on: [JSON-RPC](https://www.jsonrpc.org/specification#parameter_structures) > If present, parameters...

Unknown configuration tag

Hi @gdobreva, thanks for the report. Unfortunately there is no recent release of `ocaml-lsp-server` that would mute these errors for OCaml 4.13. Would you be able to migrate your project...

ERROR while compiling ocaml-lsp-server.1.20.1-4.14

Your branch with the patch is named `1.18.0-freebsd`, which suggest it is not based on the latest commit from the 4.14 branch: https://github.com/ocaml/ocaml-lsp/commits/414-LTS

ERROR while compiling ocaml-lsp-server.1.20.1-4.14

I was hoping for someone more knowledgeable about these subjects to intervene, but let's move on. Rudi has effectively much less time to dedicate to the project these days.

:MerlinOccurrencesProjectWide fails with KeyError

Thanks for the report and investigation @talex5, I am not familiar with the vim implementation, so forwarding to @Julow

Inlay hints disappear unexpectedly

Thanks for the report. This one might have been fixed already as part of https://github.com/ocaml/ocaml-lsp/pull/1515 @xvw can you confirm ?

Inlay hints disappear unexpectedly

Thanks for testing @rabiescow ! We will probably cut a release with these fixes early in June :-)

Support for inlay hints

🤔 it indeed looks like inlay-hints are not working anymore, but they should be. They should be controlled by the editor-wide settings `Editor › Inlay Hints: Enabled`.

Signature help does not work in the presence of type aliases

Thanks for the report @ddickstein, that's a quite surprising behavior... I added a test with your reproduction.

Short-paths handles `open` poorly

Thanks @liam923, I will add these to the testsuite.