Skip to content

Commit

Permalink
remove debug lines and add changelog
Browse files Browse the repository at this point in the history
  • Loading branch information
austinvalle committed Oct 27, 2023
1 parent b503028 commit fd5f36a
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 12 deletions.
5 changes: 5 additions & 0 deletions .changes/unreleased/BUG FIXES-20231027-092614.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
kind: BUG FIXES
body: Fixed malformed stdout when wrapper is enabled
time: 2023-10-27T09:26:14.675402-04:00
custom:
Issue: "367"
7 changes: 1 addition & 6 deletions dist/index1.js
Original file line number Diff line number Diff line change
Expand Up @@ -27222,19 +27222,14 @@ async function checkTerraform () {
const options = {
listeners,
ignoreReturnCode: true,
silent: true, // work around github.com/actions/toolkit#649
silent: true, // avoid printing command in stdout: https://github.com/actions/toolkit/issues/649
};
const exitCode = await exec(pathToCLI, args, options);

// Pass-through stdout/err as `exec` won't due to `silent: true` option
process.stdout.write(stdout.contents);
process.stderr.write(stderr.contents);

core.debug(`Terraform exited with code ${exitCode}.`);
core.debug(`stdout: ${stdout.contents}`);
core.debug(`stderr: ${stderr.contents}`);
core.debug(`exitcode: ${exitCode}`);

// Set outputs, result, exitcode, and stderr
core.setOutput('stdout', stdout.contents);
core.setOutput('stderr', stderr.contents);
Expand Down
7 changes: 1 addition & 6 deletions wrapper/terraform.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,19 +34,14 @@ async function checkTerraform () {
const options = {
listeners,
ignoreReturnCode: true,
silent: true, // work around github.com/actions/toolkit#649
silent: true, // avoid printing command in stdout: https://github.com/actions/toolkit/issues/649
};
const exitCode = await exec(pathToCLI, args, options);

// Pass-through stdout/err as `exec` won't due to `silent: true` option
process.stdout.write(stdout.contents);
process.stderr.write(stderr.contents);

core.debug(`Terraform exited with code ${exitCode}.`);
core.debug(`stdout: ${stdout.contents}`);
core.debug(`stderr: ${stderr.contents}`);
core.debug(`exitcode: ${exitCode}`);

// Set outputs, result, exitcode, and stderr
core.setOutput('stdout', stdout.contents);
core.setOutput('stderr', stderr.contents);
Expand Down

0 comments on commit fd5f36a

Please sign in to comment.