Skip to content
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

use fzf instead of ctrl-p and ack.vim instead of ag.vim #9

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ryanong
Copy link
Contributor

@ryanong ryanong commented Jun 16, 2016

So much faster and better features

This ignores .gitignore correctly over ctrlp-py-matcher

@attack
Copy link
Owner

attack commented Jun 20, 2016

@ryanong Thanks Ryan, I will test this config for a bit before merging.

@ryanong
Copy link
Contributor Author

ryanong commented Jun 20, 2016

The only issue with this branch is that Ag does not support searching within a path, but the responsiveness is much better.

@ryanong ryanong force-pushed the fzf branch 3 times, most recently from e57c437 to 909146c Compare September 18, 2017 17:07
@ryanong ryanong changed the title use fzf instead of ctrl-p and ag.vim use fzf instead of ctrl-p and ack.vim instead of ag.vim Sep 18, 2017
@ryanong
Copy link
Contributor Author

ryanong commented Sep 18, 2017

@attack fixed the bugs that were originally there. FZF is amazing you should really try it.
And replaced ag.vim with ack.vim because ag.vim is no longer supported (https://github.com/rking/ag.vim)

This also supports rg (ripgrep: a rust alternative to ag/ack) but it takes some configuring to get working as fast as ag. specifically you need to have better .rgingore file than a .agignore file as ag will automatically skip some binary files that rg won't.

@attack
Copy link
Owner

attack commented Feb 1, 2018

@ryanong Finally getting around to this.

I am doing this in two phases

  1. allow/prefer use of rg over ag
  2. replace ctrl-p

I am also testing alternatives to ack.vim (ie: ferret).

Regardless, this should result in achieving your initial goal.

@ryan-plated
Copy link

ooooo ferret looks nice going to try it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants