packer.nvim icon indicating copy to clipboard operation
packer.nvim copied to clipboard

fix: break sequencing

Open ifplusor opened this issue 2 years ago • 5 comments

close #865

ifplusor avatar Apr 11 '22 04:04 ifplusor

@wbthomason can we merge it? Currently using ifplusor branch to work with vscode-neovim plugin.

chrisands avatar Jun 13 '22 20:06 chrisands

@ifplusor @chrisands Sorry for taking so long to get to this! I'm mostly OK with merging, but I don't entirely understand what skip_lazy is meant to do - can you explain the intent in the context of #865?

wbthomason avatar Jul 26 '22 07:07 wbthomason

@ifplusor @chrisands Sorry for taking so long to get to this! I'm mostly OK with merging, but I don't entirely understand what skip_lazy is meant to do - can you explain the intent in the context of #865?

If a plugin is set to lazy load and the condition(by keys, by commands, by events, etc.) has not yet fired, skip it.

ifplusor avatar Jul 26 '22 08:07 ifplusor

However, I only check commands and keys, not events.

ifplusor avatar Jul 26 '22 08:07 ifplusor

Ok, thanks - that was the source of my confusion (i.e. why just keys and commands). We should make that check for any remaining untriggered lazy loading, I believe.

wbthomason avatar Jul 26 '22 16:07 wbthomason