Skip to content

Commit

Permalink
fix(assign-pr-reviewers): stop returning response from axios (#338)
Browse files Browse the repository at this point in the history
* fix(assign-pr-reviewers): stop returning response from axios

* stop returning inside notify-user.ts
  • Loading branch information
danadajian authored Feb 21, 2023
1 parent 2f61b0f commit ca4d3b1
Show file tree
Hide file tree
Showing 6 changed files with 7 additions and 7 deletions.
4 changes: 2 additions & 2 deletions dist/153.index.js

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

2 changes: 1 addition & 1 deletion dist/153.index.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion dist/676.index.js

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

2 changes: 1 addition & 1 deletion dist/676.index.js.map

Large diffs are not rendered by default.

2 changes: 1 addition & 1 deletion src/helpers/assign-pr-reviewers.ts
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ export const assignPrReviewers = async ({
});

if (slack_webhook_url) {
return map(
await map(
assignees,
async assignee =>
notifyUser({
Expand Down
2 changes: 1 addition & 1 deletion src/utils/notify-user.ts
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,7 @@ export const notifyUser = async ({ login, pull_number, slack_webhook_url }: Noti
} = await octokit.pulls.get({ pull_number, ...context.repo });

try {
return axios.post(slack_webhook_url, {
await axios.post(slack_webhook_url, {
assignee: email,
title,
html_url,
Expand Down

0 comments on commit ca4d3b1

Please sign in to comment.