Ulysse
Ulysse
Following the discussion in https://github.com/ocaml/dune/pull/10324 , adding a `--context CONTEXT` flag to `dune ocaml-merlin` has the advantage of not requiring changes to the protocol. This means in particular that no...
I made a fairly minimal reproduction: ```shell #!/bin/sh cat >dune-project odoc.2.3.0 will be installed from cache # -> ocamlformat.0.26.1 will be installed from cache dune build ################### #### No issue...
> I have been using [`topiary`](https://topiary.tweag.io) to for formatting instead of `ocamlformat` because `topiary` can support tabs like OCaml the language can support tabs. I’m getting an error on startup...
That's definitively a Merlin issue. We already fixed issues with that feature (my guess is that it's the `let* (), () = Lwt.both sleeper leaker in` expression that breaks merlin),...
@ELLIOTTCABLE I just made a small project with your file, OCaml 4.14, ocaml-lsp-server 1.17.0 and merlin-lib 4.13-414 and I am not able to reproduce the crash with vscode.
> Okay, I just tried to replicate the above. It seems to be an issue with `int` aliases. I changed the type to `float` and `incr` displays the "expected" type:...
I agree that having the client send the configuration looks like the simplest fix, but it's a shame that the solution would be client-specific. It's very surprising that the protocol...
> > This pull model replaces the old push model were the client signaled configuration change via an event. If the server still needs to react to configuration changes (since...
> I'll let @voodoos merge ack, I will try to release Merlin so that we can properly run the testsuite before merging.
Thanks @PizieDust, I will merge once the checks finish.