PR Review Comments Notifications:

Scott Basse · Scott Basse · commit 7a9202fcbca7 · 2025-10-01T08:48:18.000+02:00
- Don't notify about new review comments once the PR is mergeable as a notification with link attached is already send out for this
- Send notification to original PR author and not the author of the slack message
- Make PR title bold in message
diff --git a/command/pullrequest/pull_request.go b/command/pullrequest/pull_request.go
@@ -378,7 +378,7 @@ func (c command) GetHelp() []bot.Help {
 }
 
 func (c *command) notifyNewReviewComments(prw *pullRequestWatch) {
-	if !c.cfg.Notifications.NewReviewComments.IsEnabled() {
+	if !c.cfg.Notifications.NewReviewComments.IsEnabled() || prw.DidNotifyMergeable {
 		return
 	}
 
@@ -388,5 +388,5 @@ func (c *command) notifyNewReviewComments(prw *pullRequestWatch) {
 
 	prw.SavedLatestReviewCommentTimestamp = prw.PullRequest.LatestReviewCommentsTimestamp
 
-	c.sendPrivateMessage(prw.Author, "PR '%s' \nNew review comments were added: %s", prw.PullRequest.Name, getPRLinkMessage(prw))
+	c.sendPrivateMessage(prw.PullRequest.Author, "PR *'%s'* \nNew review comments were added: %s", prw.PullRequest.Name, getPRLinkMessage(prw))
 }

Original file line number	Diff line number	Diff line change
`@@ -378,7 +378,7 @@ func (c command) GetHelp() []bot.Help {`
`378`	`378`	`}`
`379`	`379`
`380`	`380`	`func (c command) notifyNewReviewComments(prw pullRequestWatch) {`
`381`		`- if !c.cfg.Notifications.NewReviewComments.IsEnabled() {`
	`381`	`+ if !c.cfg.Notifications.NewReviewComments.IsEnabled() \|\| prw.DidNotifyMergeable {`
`382`	`382`	`return`
`383`	`383`	`}`
`384`	`384`
`@@ -388,5 +388,5 @@ func (c command) notifyNewReviewComments(prw pullRequestWatch) {`
`388`	`388`
`389`	`389`	`prw.SavedLatestReviewCommentTimestamp = prw.PullRequest.LatestReviewCommentsTimestamp`
`390`	`390`
`391`		`- c.sendPrivateMessage(prw.Author, "PR '%s' \nNew review comments were added: %s", prw.PullRequest.Name, getPRLinkMessage(prw))`
	`391`	`+ c.sendPrivateMessage(prw.PullRequest.Author, "PR '%s' \nNew review comments were added: %s", prw.PullRequest.Name, getPRLinkMessage(prw))`
`392`	`392`	`}`