Fix return Promises so each step runs in sequence

[xoob]

Return the Promise of each executeCommand() call to PromiseSeries, so that commands are executed in sequence, not in parallel.

[josenicomaia]

+1

[Spongman]

please merge this. this extension is pretty much broken without this.

[josenicomaia]

@Spongman Indeed

[Spongman]

also, the 'custom delay' patch is unnecessary with this patch.

[mizagog]

why is it not in...?

[therealpeterhua]

Is this extension abandoned? This fix is pretty core, why no merge?

[clayms]

This worked well for me and is easy to understand. Simply add return to the beginning of lines 34 and 38.

Current:

macros/extension.js

Line 34 in 96b0d7b

vscode.commands.executeCommand(action.command, action.args);

macros/extension.js

Line 38 in 96b0d7b

vscode.commands.executeCommand(action);

Fixed:

macros/extension.js

Line 34 in de2633d

return vscode.commands.executeCommand(action.command, action.args);

macros/extension.js

Line 38 in de2633d

return vscode.commands.executeCommand(action);

[jpotterm]

Since it's been three years with no merge I found this fork that fixes this: https://github.com/ctf0/macros

[tobiashochguertel]

@geddski how is it about this Pull Request? Can we may work on this this weekend and merge and release a version 1.3?

[Ashark]

This worked well for me and is easy to understand. Simply add return to the beginning of lines 34 and 38.

Current:

macros/extension.js

Line 34 in 96b0d7b

vscode.commands.executeCommand(action.command, action.args);

macros/extension.js

Line 38 in 96b0d7b

vscode.commands.executeCommand(action);

Fixed:

macros/extension.js

Line 34 in de2633d

return vscode.commands.executeCommand(action.command, action.args);

macros/extension.js

Line 38 in de2633d

return vscode.commands.executeCommand(action);

I also am affected by the issue. See https://stackoverflow.com/questions/72153504/prevent-asynchronous-order-of-command-sequence-execution-in-vs-codium

I added these two return in the ~/.vscode/extensions/geddski.macros-1.2.1/extension.js, but it did not work for me. It still always execute commands in parallel. Any advise?

[Ashark]

Since it's been three years with no merge I found this fork that fixes this: https://github.com/ctf0/macros

@jpotterm How to install that? It seems it has not been published in marketplace?