Skip to content
This repository has been archived by the owner on Sep 20, 2024. It is now read-only.

Commit

Permalink
refactor: send messages error to stderr (#100)
Browse files Browse the repository at this point in the history
* chore: Update Packages

* refactor: send messages error to stdout

* Update delaySending.js

* Update ReleaseCommunicationFacade.js

Co-authored-by: therynamo <[email protected]>
  • Loading branch information
AndersDJohnson and therynamo authored Mar 9, 2021
1 parent 3e7f0d0 commit 785704d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions src/facades/ReleaseCommunicationFacade.js
Original file line number Diff line number Diff line change
Expand Up @@ -82,8 +82,8 @@ class ReleaseCommunication {
// Should allow execution to continue for repositories that use a different merge method.
try {
prNumbersByCommit = await Promise.all(nonSquashedPRs.map(async commit => searchIssuesByCommitHandler(commit)));
} catch (e) {
console.log('Unable to retrieve issues by commit. Check if API limit was exceeded.', e);
} catch (error) {
console.error('Unable to retrieve issues by commit. Check if API limit was exceeded.', error);
}

// There should only ever be one issue for a commit
Expand Down
2 changes: 1 addition & 1 deletion src/utils/delaySending.js
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ const delaySending = async function delaySending(sender, items, ms) {
try {
sender(items[i]);
} catch (error) {
console.log(error);
console.error(error);
}
}
};
Expand Down
2 changes: 1 addition & 1 deletion src/utils/sendMessages.js
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ const sendDelayedMessages = async function sendMessages(sender, blocks) {
try {
sender(blockChunk);
} catch (error) {
console.log(error);
console.error(error);
}
}
};
Expand Down

0 comments on commit 785704d

Please sign in to comment.