Ulysse
Ulysse
cc @let-def
What is the status on this ? @rgrinberg has your patch been in used in released versions of ocaml-lsp yet ? Would you mind opening a PR with its latest...
@rgrinberg, @let-def, any news on this ?
Currently the `rename` command is implemented in the editor plugin itself. The editor calls Merlin's `occurrences` command which answers with a list of positions which is used by the plugin...
You are right, and the last one looks tricky. Maybe this should be a Merlin's command and not an editor hack. I guess this is what @let-def meant in #610
Not investigated yet, but this seems related to https://github.com/ocaml/dune/issues/4169
Thank you for the report! I added a test illustrating the issue. A major rework of that feature is planned in a not-so-distant future...
Yes, that is part of the plan :-)
I feel like the current behaviour is actually correct: When calling jump-to-definition at that (`|`) point: ```ocaml module M = struct type t = { foo : int } let...
Yes that makes sense. I will update the test before merging.