Skip to content

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
@sxyazi sxyazi force-pushed the main branch 3 times, most recently from d1efbb8 to a0ab614 Compare June 16, 2025 13:25
@sxyazi sxyazi force-pushed the main branch 4 times, most recently from 35d9907 to 2768fd2 Compare June 27, 2025 09:44
@sxyazi sxyazi force-pushed the main branch 3 times, most recently from 9f077f9 to 60a2382 Compare July 14, 2025 15:08
@Shallow-Seek
Copy link

I am writing a plugin that allows you to configure the search command fd, rg, etc., and all fzf options in one config file beside the main.lua so you can populate different fzf utilities in yazi. I will post it in a few days.

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
3 participants