perf(trouble-nvim): avoid eagerly loading trouble when configuring snacks integration#1759
perf(trouble-nvim): avoid eagerly loading trouble when configuring snacks integration#1759ALVAROPING1 wants to merge 1 commit intoAstroNvim:mainfrom
Conversation
Review ChecklistDoes this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist: Proper conventional commit scoping:
|
|
ngl this looks hedious as fuck to me and not something I'm particularly eager to maintain when its not clear what it does on a first look. wdyt @azdanov |
|
Hey. I don't think this is a part of a "community" plugin. If something breaks or becomes incompatible in the future then no one will be able to understand what was the issue. It's more of a personal dotfile thing. |
|
Gonna agree with @azdanov. Unless you come up with a version of this that's easier to maintain. This is not good enough as is. |
|
This is the simplest way i could come up with, so i will keep it in my personal config 👍 |
📑 Description
The previous method eagerly required trouble while configuring snacks to get trouble's picker actions. New method only requires it when indexing the actions table. Sadly snacks still loads trouble the first time any picker is opened rather than when the action is first triggered, but it's better than loading it at startup