Skip to content

Commit

Permalink
fix: await a promise instead of not
Browse files Browse the repository at this point in the history
  • Loading branch information
mlg87 committed May 18, 2022
1 parent d293270 commit ec828d4
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/actions/handleCommitPush.ts
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ export const handleCommitPush = async (): Promise<void> => {
const previousReviewers = res.data.map((review) => review!.user!.login);
const distinctPreviousReviewers = [...new Set(previousReviewers)];
const baseMessage = `new code has been committed since your review of <${pull_request._links.html.href}|*PR ${pull_request.number}*>, please review the updates.`;
const usersToAtString = createUsersToAtString(distinctPreviousReviewers);
const usersToAtString = await createUsersToAtString(distinctPreviousReviewers);
const text = `${usersToAtString} ${baseMessage}`;
const threadUpdateRes = await slackWebClient.chat.postMessage({
channel: channelId,
Expand Down

0 comments on commit ec828d4

Please sign in to comment.