-
Notifications
You must be signed in to change notification settings - Fork 233
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
Merlin emacs mode does not support project-wide occurrences #1740
Comments
@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 testsuite:
These are still moving parts, if by mistake the latest versions of these tools don't work together you can follow the instructions here for a working 4.14-base version of the feature: https://discuss.ocaml.org/t/ann-preview-play-with-project-wide-occurrences-for-ocaml/13814 The emacs mode should work regardless :-) |
With the support for project wide occurrences approaching, we need to update Merlin clients.
Some attempts was made to update the emacs mode that are buggy right now: it breaks the standard local occurrences queries, doesn't always show the name of the file and sometimes returns error (when the list is too big ?).
Working on this requires an experimental setup with:
The current, fragile patch can be found here: voodoos@141fe8d
The text was updated successfully, but these errors were encountered: