Ulysse comments

Results 310 comments of


                                            Ulysse

Documentation query

> I likewise don't have an objection to an option to overwrite the format specified at initialization, but I do expect a given editor will always want the response in...

Merlin emacs mode does not support project-wide occurrences

@xvw To tests your changes on a small scale you won't need the custom dune: you can build the index by hand and instruct Merlin to use it: From [the...

`case-analysis` fails to load module

Thanks for the report and the reproduction @liam923 It's an interesting corner-case... cc @xvw since it concerns destruct (even if not the core of it's logic).

ocamllsp is never stopped, new ones started each time

Thanks for your report @vphantom. So far I was not able to reproduce so far with the official vscode version on macos. I tried opening / closing a window with...

Optional argument include by `case-analysis`

Thanks for the report @liam923 ! cc our destruct expert @xvw

Go-to-definition does not work on custom operators

Here is how I would proceed: - Write a test illustrating the wrong behavior (similar as https://github.com/ocaml/merlin/blob/main/tests/test-dirs/locate/issue949.t) - Such a test would already be welcome as a contribution - Then...

Improve location of some custom operators

Thanks for the patch @WardBrian. I will review it when I get some spare cycles (it might take some time..).

Jump to incorrect module

Thanks for the precise reproduction, I am adding it to the test suite in #1750

Jump to incorrect module

I also pushed a potential fix for the issue in the reproduction PR.

Add ocamlformat-mlx for .mlx files

@davesnx I am not familiar with ocamlformat-mlx, but overall this looks correct. However the CI is failing with a non-exhaustive pattern matching error, could you fix that ?