uses stderr content to define error

Author: yunchengyang515

wrapper/dist/index.js

@@ -4209,7 +4209,7 @@ async function checkStackQL () {
   core.setOutput('stderr', stderr.contents);
   core.setOutput('exitcode', exitCode.toString(10));
 
-  if (exitCode === 0 || exitCode === 2 || !stderr.contents) {
+  if ((exitCode === 0 || exitCode === 2) && !stderr.contents) {
     // 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

wrapper/stackql.js

@@ -31,10 +31,10 @@ async function checkStackQL () {
     ignoreReturnCode: true
   };
   const exitCode = await exec(pathToCLI, args, options);
-  console.log(`StackQL exited with code ${exitCode}.`);
-  console.log(`stdout: ${stdout.contents}`);
+  core.debug(`StackQL exited with code ${exitCode}.`);
+  core.debug(`stdout: ${stdout.contents}`);
   console.log(`stderr: ${stderr.contents}`);
-  console.log(`exitcode: ${exitCode}`);
+  core.debug(`exitcode: ${exitCode}`);
 
   // Set outputs, result, exitcode, and stderr
   core.setOutput('stdout', stdout.contents);