search

ocaml / merlin

Context sensitive completion for OCaml in Vim and Emacs
https://ocaml.github.io/merlin/
MIT License
1.57k stars 233 forks source link

Jump to definition from the dropdown #76

Closed roshanjames closed 10 years ago

roshanjames commented 11 years ago

This a feature request for the emacs mode: It would be nice to be able to jump to a location from an item displayed in the dropdown without having to insert it in the current buffer and subsequently doing merlin-locate on it.

asmanur commented 11 years ago

Hi,

This has been implemented in emacs-dev which should be merged soon (5894130). On sam. 17/août 2013 (20:42), Roshan James wrote:

This a feature request for the emacs mode: It would be nice to be able to jump to a location from an item displayed in the dropdown without having to insert it in the current buffer and subsequently doing merlin-locate on it.

Reply to this email directly or view it on GitHub: https://github.com/def-lkb/merlin/issues/76

roshanjames commented 11 years ago

Can you guys merge emacs-dev into master? I think the branch has accumulated a few useful patches by now which are worth testing out.

trefis commented 11 years ago

2013/8/19 Roshan James notifications@github.com

Can you guys merge emacs-dev into master? I think the branch has accumulated a few useful patches by now which are worth testing out.

Well, that's kind of the point of the separate branch. The changes are on their branch so people can test them and make sure they don't break anything. Once we're confident they don't, we'll merge. :)

roshanjames commented 11 years ago

Ok :) I'll wait.

On Aug 19, 2013, at 10:22 AM, Thomas Refis notifications@github.com wrote:

2013/8/19 Roshan James notifications@github.com

Can you guys merge emacs-dev into master? I think the branch has accumulated a few useful patches by now which are worth testing out.

Well, that's kind of the point of the separate branch. The changes are on their branch so people can test them and make sure they don't break anything. Once we're confident they don't, we'll merge. :) — Reply to this email directly or view it on GitHub.

trefis commented 10 years ago

That has been on master for a while, and it's definitely working. Enjoy :)