Merge pull request #24 from stackql/feature/updates

stderr fix

README.md

@@ -17,7 +17,7 @@ Check the "Use GitHub Provider" step in [setup-stackql.yml](.github/workflows/se
 
 ```yaml
     - name: setup StackQL
-      uses: stackql/setup-stackql@v1.2.0
+      uses: stackql/setup-stackql@v2.0.0
       with:
         use_wrapper: true
 
@@ -34,7 +34,7 @@ Check the "Use Google Provider" step in [setup-stackql.yml](.github/workflows/se
 
 ```yaml
     - name: setup StackQL
-      uses: stackql/setup-stackql@v1.2.0
+      uses: stackql/setup-stackql@v2.0.0
       with:
         use_wrapper: true
 

package.json

@@ -1,6 +1,6 @@
 {
   "name": "setup-stackql",
-  "version": "1.2.0",
+  "version": "2.0.0",
   "description": "",
   "main": "index.js",
   "scripts": {

wrapper/dist/index.js

@@ -4208,16 +4208,17 @@ async function checkStackQL () {
   core.setOutput('stdout', stdout.contents);
   core.setOutput('stderr', stderr.contents);
 
-  if ((exitCode === 0 || exitCode === 2) && !stderr.contents) {
+  if (exitCode === 0 || exitCode === 2) {
     // A exitCode of 0 is considered a success
     // An exitCode of 2 may be returned when the '-detailed-exitcode' option
     // is passed to plan. This denotes Success with non-empty
     // diff (changes present).
     return;
   }
 
-  // A non-zero exitCode is considered an error
-  core.setFailed(`StackQL exited with error: ${stderr.contents}.`);
+  // If we reach this point, it means that the exitCode was neither 0 nor 2,
+  // which denotes an error. We can still use stderr.contents to provide more information about the error.
+  core.setFailed(`StackQL exited with error code ${exitCode}. Details: ${stderr.contents}.`);
 })();
 
 })();

wrapper/stackql.js

@@ -40,14 +40,15 @@ async function checkStackQL () {
   core.setOutput('stdout', stdout.contents);
   core.setOutput('stderr', stderr.contents);
 
-  if ((exitCode === 0 || exitCode === 2) && !stderr.contents) {
+  if (exitCode === 0 || exitCode === 2) {
     // A exitCode of 0 is considered a success
     // An exitCode of 2 may be returned when the '-detailed-exitcode' option
     // is passed to plan. This denotes Success with non-empty
     // diff (changes present).
     return;
   }
 
-  // A non-zero exitCode is considered an error
-  core.setFailed(`StackQL exited with error: ${stderr.contents}.`);
+  // If we reach this point, it means that the exitCode was neither 0 nor 2,
+  // which denotes an error. We can still use stderr.contents to provide more information about the error.
+  core.setFailed(`StackQL exited with error code ${exitCode}. Details: ${stderr.contents}.`);
 })();