-
Notifications
You must be signed in to change notification settings - Fork 10
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
[advice] Make fzf more robust #16
Comments
Sorry for the long wait in responding to this issue, and thank you for the recommendation: definitely a good idea. I will work on it in the next release :) |
This PR addresses the issue: would you be so kind to help me test it to make sure there are no particular corner cases where the changes may instead behave otherwise? Ideally you could clone the repository and install the local branch
and use it off: so far it seems all is good, but four eyes are better than two! |
Thanks. I have used this version, and since now I haven't found any incorrect behaviour. |
Released in v0.3.3. Just |
I advice change 12
fzf
s tofzf -d' '
s. Because fzf will be affected by$FZF_DEFAULT_OPT
. If user use-d:
or other-d
, it will make fzf work abnormally.Thanks.
The text was updated successfully, but these errors were encountered: