refactor(commands): move util function to toolbox

hinell commented 1 year ago

I actually wanted to include this patch into #415 PR, but decided to go separately.

How to Test

N/A

I have tested the following:

[x] Triggering keymaps from legendary.nvim in all modes (normal, insert, visual)
[x] Creating keymaps via legendary.nvim, then triggering via the keymap in all modes (normal, insert, visual)
[x] Triggering commands from legendary.nvim in all modes (normal, insert, visual)
[x] Creating commands via legendary.nvim, then running the command manually from the command line
[x] augroup/autocmds created through legendary.nvim work correctly

mrjones2014 commented 1 year ago

What's the purpose of this PR?

hinell commented 1 year ago

@mrjones2014 Refactoring. That function might be used elsewhere, so it's logical to keep it in tool box.

mrjones2014 commented 1 year ago

That function might be used elsewhere

But it isn't currently. We can move it if/when it's needed.

mrjones2014 commented 1 year ago

Toolbox is supposed to be more for end-user utilities, this is an internal parser.