Merge pull request #136 from zowe-actions/fix/windows-syncpack

Fix syncpack update not working on Windows
zowe-actions · Feb 23, 2024 · 61cfdfa · 61cfdfa
2 parents 0838328 + 616508a
commit 61cfdfa
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/dist/run-script.js b/dist/run-script.js
diff --git a/script/scripts/npmUpdate.ts b/script/scripts/npmUpdate.ts
@@ -127,8 +127,9 @@ export default async function (context: IContext): Promise<void> {
             changedFiles.push("**/package.json");
             const dependencyList = [...Object.keys(dependencies), ...Object.keys(devDependencies)];
 
+            // See https://en.wikipedia.org/wiki/Escape_character#Windows_Command_Prompt for why we need '^'
             await exec.exec("npx", ["-y", "--", "syncpack@8", "fix-mismatches", "--dev", "--prod", "--filter",
-                dependencyList.join("|")]);
+                dependencyList.join(process.platform === "win32" ? "^|" : "|")]);
             await exec.exec("git", ["checkout", lockfilePath]);
             await exec.exec("npm", ["install"]);
         }