Skip to content

Commit 6fe40fa

Browse files
authored
email notifications now includes a link to the PR itself (#220)
1 parent cdff8f2 commit 6fe40fa

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

notification.js

+2-2
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ exports.notifyContacts = async function (gh, pr, status, mailer, emailConfig, st
2222
from: emailConfig.from,
2323
to: actualEmails.join(","),
2424
cc: emailConfig.cc.join(","),
25-
subject: "IPR check failed for PR #" + pr.num+ " on " + pr.fullName,
26-
text: status.payload.description + "\n\n See " + status.payload.target_url
25+
subject: `IPR check failed for PR #${pr.num} on ${pr.fullName}`,
26+
text: `${status.payload.description}\n\nSee ${status.payload.target_url}\nand https://github.com/${pr.fullName}/pull/${pr.num}`
2727
});
2828
for await (let user of pr.unaffiliatedUsers
2929
.map(u => doAsync(store).getUser(u))) {

0 commit comments

Comments
 (0)