From 7a6c51af1ccea6060c3b8f0ce3bfb753d41bd337 Mon Sep 17 00:00:00 2001 From: rtm516 Date: Sat, 25 Oct 2025 21:14:42 +0100 Subject: [PATCH 1/2] Remove touch command from yarn installation This fixes an issue with installing on windows due to touch not being available --- npm/deepinstall.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/npm/deepinstall.js b/npm/deepinstall.js index 802d83863..ae1b591f4 100644 --- a/npm/deepinstall.js +++ b/npm/deepinstall.js @@ -44,7 +44,7 @@ async.series([ command = 'yarn install --production --frozen-lockfile'; } else { - command = 'touch yarn.lock && yarn workspaces focus --all --production' + command = 'yarn workspaces focus --all --production' } break; case 'pnpm': From 53a0350a989d8572a3e5750d320340fb8eb032cb Mon Sep 17 00:00:00 2001 From: rtm516 Date: Sun, 28 Dec 2025 21:28:18 +0000 Subject: [PATCH 2/2] Refactor codegen installation loop to use for..of Fixes an issue where the next is called 2x due to return not being handled as expected --- npm/deepinstall.js | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/npm/deepinstall.js b/npm/deepinstall.js index ae1b591f4..ae6e4add1 100644 --- a/npm/deepinstall.js +++ b/npm/deepinstall.js @@ -68,8 +68,7 @@ async.series([ }, function (next) { var codegens = getSubfolders(PATH_TO_CODEGENS_FOLDER); - codegens.forEach((codegen) => { - + for (var codegen of codegens) { shell.cd(codegen.path); var commandOut; @@ -82,7 +81,7 @@ async.series([ return next(commandOut.stderr); } console.log(commandOut.stdout); - }); + }; }], (err) => { if (err) { console.error(err);