dajva / rg.el

Emacs search tool based on ripgrep
https://rgel.readthedocs.io
GNU General Public License v3.0
484 stars 41 forks source link

add rg-dwim-current-file to narrow rg-dwim even more. #55

Closed kidd closed 5 years ago

kidd commented 5 years ago

Following the trend of c-u meaning "more focused", this commit implements c-u c-u rg-dwim as even narrower search, searching just in the current file.

Still have to figure out how to run the tests correctly in my box.

coveralls commented 5 years ago

Coverage Status

Coverage increased (+0.03%) to 84.431% when pulling 6470d52290c6d398feae8fad116b85b76cd014ae on kidd:add-rg-dwim-current-file into f09df8858fc33f94f7e3e5c7c91e58c2defc3760 on dajva:master.

coveralls commented 5 years ago

Coverage Status

Coverage increased (+3.7%) to 88.119% when pulling a70dcd07e87254cbd7cb1266fa76b2745b239ceb on kidd:add-rg-dwim-current-file into f09df8858fc33f94f7e3e5c7c91e58c2defc3760 on dajva:master.

kidd commented 5 years ago

Hello, and thanks for the review.

Totally right on all points. The whole thing wasn't "ready" for merge yet. And good catch on 2.

I think I addressed all of them. If you have further ideas on making the wording clearer, feel free to merge and edit the docstrings right away.

dajva commented 5 years ago

LGTM. Merged. Thanks.