Skip to content

Commit 147522b

Browse files
authored
Merge pull request #2212 from Urgau/mcp-impv-acceptance-comment
Improve clarity of the major change acceptance comment
2 parents 1b5a63a + 9d1e864 commit 147522b

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/handlers/major_change.rs

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -705,7 +705,8 @@ async fn try_accept_mcp(
705705

706706
format!(
707707
r"
708-
Progress of this major change will be tracked in the tracking issue at {}#{}.
708+
> [!NOTE]
709+
> Further progress is now tracked over at {}#{}.
709710
",
710711
&issue_repo, tracking_issue.number
711712
)
@@ -717,11 +718,12 @@ Progress of this major change will be tracked in the tracking issue at {}#{}.
717718
.post_comment(
718719
&ctx.github,
719720
&format!(
720-
r"The final comment period is now complete, this major change is now **accepted**.
721-
{tracking_issue_text}
722-
As the automated representative, I would like to thank the author for their work and everyone else who contributed to this major change proposal.
721+
r"The final comment period is now complete, this major change is now **accepted**. :tada:
723722
724-
*If you think this major change shouldn't have been accepted, feel free to remove the `{}` label and reopen this issue.*",
723+
As the automated representative, I want to express gratitude to the author for their work and everyone else who contributed to this major change proposal.
724+
725+
*If you think this major change shouldn't have been accepted, feel free to remove the `{}` label and reopen this issue.*
726+
{tracking_issue_text}",
725727
&config.accept_label,
726728
),
727729
)

0 commit comments

Comments
 (0)