Moved common functionality from runCustomPRCommand() & runCustomIssueCommand() into new runCustomCommand() function
How did you test this change?
Set and ran rudimentary custom commands in each context. Could probably do with some more robust testing though tbh.
I've left an empty input map still initialised in the new function, and merged in the specific data from the calls from different contexts, as I'm not sure what common data might want to be made available common to different contexts.
Following the calls back, could drop runCustomPRCommand() & runCustomIssueCommand() too I suppose, unless there's likely to be extra logic there that doesn't want to be polluting executeKeybinding().
Summary
Moved common functionality from
runCustomPRCommand()
&runCustomIssueCommand()
into newrunCustomCommand()
functionHow did you test this change?
Set and ran rudimentary custom commands in each context. Could probably do with some more robust testing though tbh.
I've left an empty
input
map still initialised in the new function, and merged in the specific data from the calls from different contexts, as I'm not sure what common data might want to be made available common to different contexts.