Jack Feser
Jack Feser
I suspect that users will want this to be configurable. I could see how novice users might find this information helpful, but it's noisy. https://github.com/ocaml/ocaml-lsp/issues/1198 suggests go-to links for types....
I can reproduce the issue with `let`. I agree that we shouldn't be producing an error in these cases. Are you seeing this issue in other contexts?
A few thoughts: - Right now, merlin returns Not_in_env for syntax like `let`, `match`, `begin`, etc. It should return Invalid_context instead, which we handle appropriately. This is a merlin bug....
Can you give an example of the issue with signature help?
I can't reproduce the crash. Could you make sure that it happens on the latest version?
See also #1138
Also running into this. It would be very convenient for dune to be able to promote over nonexistent files.