refactor(lazydocker): replace keybindings table by function instead

cristobalgvera commented 3 weeks ago

📑 Description

Standardize the keybinding of lazydocker.nvim to be a function instead of an opts table.

Also, it was removed the unnecesary opts.

github-actions[bot] commented 3 weeks ago

Does this PR follow the [Contribution Guidelines](development guidelines)? Following is a partial checklist:

Proper conventional commit scoping:

If you are adding a new plugin, the scope would be the name of the category it is being added into. ex. feat(utility): added noice.nvim plugin
If you are modifying a pre-existing plugin or pack, the scope would be the name of the plugin folder. ex. fix(noice-nvim): fix LSP handler error
[x] Pull request title has the appropriate conventional commit type and scope where the scope is the name of the pre-existing directory in the project as described above
[x] README is properly formatted and uses fenced in links with <url> unless they are inside a [title](url)
[x] Entry returns a single plugin spec with the new plugin as the only top level spec (not applicable for recipes or packs).
[x] Proper usage of opts table rather than setting things up with the config function.
[x] Proper usage of specs table for all specs that are not dependencies of a given plugin (not applicable for recipes or packs).

Uzaaft commented 2 weeks ago

I dont see why would we do this?

cc @AstroNvim/astrocommunity-maintainers

ALameLlama commented 2 weeks ago

It would be nice to have it "Standardize" so all plugins use the same method imo