-
Notifications
You must be signed in to change notification settings - Fork 117
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Issue with doc on constructors #1211
Comments
This is probably an upstream issue in Merlin. |
ocaml --version
The OCaml toplevel, version 5.0.0 ocamllsp --version
1.16.2 merlin-lib 4.9-500 Merlin's libraries |
@voodoos should we move this issue to Merlin repo? Also, would creating a test for this be helpful? After a quick look, it seems that adding something to https://github.com/ocaml/merlin/tree/master/tests/test-dirs/document might be the right place? |
Actually, there is already a similar test: https://github.com/mheiber/merlin/blob/master/tests/test-dirs/document/issue1513.t I expect some of these issues to be fixed when moving to OCaml 5.2 which should allow more precise determination of comments. |
without params:
Screen.Recording.2023-11-14.at.11.36.06.mov
with params:
Screen.Recording.2023-11-14.at.11.39.57.mov
The text was updated successfully, but these errors were encountered: