Only include npm version if npm exists

joshmgross · joshmgross · commit ffde5387817e · 2020-04-21T11:23:19.000-04:00
diff --git a/dist/index.js b/dist/index.js
@@ -15228,8 +15228,11 @@ function run() {
             const nodeVersion = child_process_1.default.execSync(`${nodePath} --version`);
             console.log(`Node Version: ${nodeVersion}`);
             const npmPath = yield io.which('npm');
-            const npmVersion = child_process_1.default.execSync(`${npmPath} --version`);
-            console.log(`npm Version: ${npmVersion}`);
+            // Older versions of Node don't include npm
+            if (npmPath) {
+                const npmVersion = child_process_1.default.execSync(`${npmPath} --version`);
+                console.log(`npm Version: ${npmVersion}`);
+            }
             const registryUrl = core.getInput('registry-url');
             const alwaysAuth = core.getInput('always-auth');
             if (registryUrl) {
diff --git a/src/setup-node.ts b/src/setup-node.ts
@@ -25,8 +25,11 @@ async function run() {
     console.log(`Node Version: ${nodeVersion}`);
 
     const npmPath = await io.which('npm');
-    const npmVersion = cp.execSync(`${npmPath} --version`);
-    console.log(`npm Version: ${npmVersion}`);
+    // Older versions of Node don't include npm
+    if (npmPath) {
+      const npmVersion = cp.execSync(`${npmPath} --version`);
+      console.log(`npm Version: ${npmVersion}`);
+    }
 
     const registryUrl: string = core.getInput('registry-url');
     const alwaysAuth: string = core.getInput('always-auth');
