Skip to content

feat: allow configuring fzf options #2716

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

CBeerta
Copy link

@CBeerta CBeerta commented May 4, 2025

Which issue does this PR close?

Closes #2715

Rationale of this PR

@sxyazi
Copy link
Owner

sxyazi commented May 4, 2025

fzf has about 100 options, and we can't possibly support each one individually.

So please consider a more general solution by forwarding any parameters provided by the user to fzf, not just --walker here.

@CBeerta
Copy link
Author

CBeerta commented May 4, 2025

Yeah, ofc. Is this better? I haven't actually ever really written any lua.

Or would you rather use a --args="--walker=hidden" style option?

@sxyazi sxyazi changed the title feat: allow setting the walker feat: allow configuring fzf options May 12, 2025
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.

Allow passing --walker options to the fzf plugin
2 participants