Skip to content

Commit

Permalink
Merge pull request #844 from github/duplicated-output
Browse files Browse the repository at this point in the history
Stop printing all output twice.
  • Loading branch information
Chris Gavin authored and GitHub committed Dec 13, 2021
2 parents ce77f88 + 1d83f2a commit 716b598
Show file tree
Hide file tree
Showing 3 changed files with 1 addition and 15 deletions.
8 changes: 0 additions & 8 deletions lib/toolrunner-error-catcher.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion lib/toolrunner-error-catcher.js.map

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

6 changes: 0 additions & 6 deletions src/toolrunner-error-catcher.ts
Original file line number Diff line number Diff line change
Expand Up @@ -29,18 +29,12 @@ export async function toolrunnerErrorCatcher(
stdout += data.toString();
if (options?.listeners?.stdout !== undefined) {
options.listeners.stdout(data);
} else {
// if no stdout listener was originally defined then we match default behavior of Toolrunner
process.stdout.write(data);
}
},
stderr: (data: Buffer) => {
stderr += data.toString();
if (options?.listeners?.stderr !== undefined) {
options.listeners.stderr(data);
} else {
// if no stderr listener was originally defined then we match default behavior of Toolrunner
process.stderr.write(data);
}
},
};
Expand Down

0 comments on commit 716b598

Please sign in to comment.