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 skim instead of fzf #61

Open
jtrv opened this issue Apr 7, 2021 · 5 comments
Open

Use skim instead of fzf #61

jtrv opened this issue Apr 7, 2021 · 5 comments

Comments

@jtrv
Copy link

jtrv commented Apr 7, 2021

Can you add support or a config option to add the rust implemented fuzzy finder "skim"? I believe it uses the same flags and syntax.

@ms-jpq
Copy link
Owner

ms-jpq commented Apr 11, 2021

ok, i need to work on this :)

@ssokolow
Copy link

skim is also usable as a library crate, which means you could just build it into sad without needing it to be installed separately.

@NightMachinery
Copy link

(Please don't remove support for fzf.)

@pickfire
Copy link

I think can just fallback to skim if fzf is not installed since skim can be used to replace fzf.

@ssokolow
Copy link

What if we have fzf installed for something that only supports it, but we prefer skim where supported? There should be an option for that.

(eg. so in-process operation can result in improved I/O access patterns to load skim compared to fzf.)

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

No branches or pull requests

5 participants