Merge pull request #375 from akv-platform/fix-error-message

Make throwing errors in setup-dotnet more informative
actions · Jan 30, 2023 · c41fd15 · c41fd15
2 parents 3cf27f1 + 0c86525
commit c41fd15
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 4 deletions.
diff --git a/dist/index.js b/dist/index.js
@@ -398,9 +398,9 @@ class DotnetCoreInstaller {
                 ignoreReturnCode: true,
                 env: process.env
             };
-            const { exitCode, stdout } = yield exec.getExecOutput(`"${scriptPath}"`, scriptArguments, getExecOutputOptions);
+            const { exitCode, stderr } = yield exec.getExecOutput(`"${scriptPath}"`, scriptArguments, getExecOutputOptions);
             if (exitCode) {
-                throw new Error(`Failed to install dotnet ${exitCode}. ${stdout}`);
+                throw new Error(`Failed to install dotnet, exit code: ${exitCode}. ${stderr}`);
             }
             return this.outputDotnetVersion(dotnetVersion.value);
         });

diff --git a/src/installer.ts b/src/installer.ts
@@ -236,13 +236,15 @@ export class DotnetCoreInstaller {
       ignoreReturnCode: true,
       env: process.env as {string: string}
     };
-    const {exitCode, stdout} = await exec.getExecOutput(
+    const {exitCode, stderr} = await exec.getExecOutput(
       `"${scriptPath}"`,
       scriptArguments,
       getExecOutputOptions
     );
     if (exitCode) {
-      throw new Error(`Failed to install dotnet ${exitCode}. ${stdout}`);
+      throw new Error(
+        `Failed to install dotnet, exit code: ${exitCode}. ${stderr}`
+      );
     }
 
     return this.outputDotnetVersion(dotnetVersion.value);