-
Notifications
You must be signed in to change notification settings - Fork 460
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
merge several "yazi.toml" files #1338
Comments
I think introducing config merging for a performance-first file manager is a bit overkill (it will require reading more files and doing more merges), and it could also cause more confusion - I've explained my thoughts in more detail in another issue here, #989 (comment) |
is it possible via the API to add previewers / loaders ? I could live with that generating a script that changes my config afterwards. |
Closing in favor of #2047 |
I'm going to lock this issue because it has been closed for 30 days. ⏳ |
Please describe the problem you're trying to solve
I generate part of my config with nix https://nixos.org/. It has several advantages, one of which is for general toggle of a plugin.
For instance I have a nix setting
yazi.enableOuchPlugin = true
that enables the yazi plugin https://github.com/ndtoan96/ouch.yazi:it puts the plugin files where expected (in ~/.config/yazi/plugins/ouch.yazi), makes "ouch" available to the plugin and enables the config advised in the plugin, i.e. the previewers and loaders.
The drawback is that generating ~/.config/yazi/yazi.toml via nix, the file becomes read-only and any change to the config becomes bothersome because it takes several seconds for nix to regenerate the config.
Would you be willing to contribute this feature?
Describe the solution you'd like
What I try to when possible do is to get the best of both worlds by merging:
Now I dont know how to do this with yazi, I dont think it's possible ?
Outside of my specific nix usecase, maybe this feature could be used by plugins to embed their own configuration ?
The possible ways to do it I could think of:
Additional context
#623 might be related
The text was updated successfully, but these errors were encountered: