Skip to content

Prevent team notifications from Maestro comment footers#5451

Merged
premun merged 1 commit intodotnet:mainfrom
premun:prvysoky/b
Nov 10, 2025
Merged

Prevent team notifications from Maestro comment footers#5451
premun merged 1 commit intodotnet:mainfrom
premun:prvysoky/b

Conversation

@premun
Copy link
Copy Markdown
Member

@premun premun commented Nov 10, 2025

Introduced in #5376

Copilot AI review requested due to automatic review settings November 10, 2025 09:23
@premun premun enabled auto-merge (squash) November 10, 2025 09:25
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR modifies the help line message in pull request comments to change how the GitHub team mention is formatted, replacing the escaped backslash approach with HTML <b> tags around the @ symbol.

  • Changed the team mention format from \\@dotnet/prodconsvcs to <b>@</b>dotnet/prodconsvcs

private readonly ILogger<PullRequestCommenter> _logger;

private const string HelpLine = $"> 💡 You may consult the [FAQ]({PullRequestBuilder.CodeFlowPrFaqUri}) for more information or tag **\\@dotnet/prodconsvcs** for assistance.";
private const string HelpLine = $"> 💡 You may consult the [FAQ]({PullRequestBuilder.CodeFlowPrFaqUri}) for more information or tag **<b>@</b>dotnet/prodconsvcs** for assistance.";
Copy link

Copilot AI Nov 10, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The use of <b>@</b> to escape the @ symbol in a Markdown context is unconventional and may render inconsistently across different platforms. The standard Markdown approach is to use a backslash escape (\\@). If the intention is to prevent GitHub from creating a team mention while keeping the text visually correct, consider using a zero-width space or other invisible character after the @ symbol (e.g., @\u200Bdotnet/prodconsvcs), or using backticks for code formatting (e.g., `@dotnet/prodconsvcs`).

Suggested change
private const string HelpLine = $"> 💡 You may consult the [FAQ]({PullRequestBuilder.CodeFlowPrFaqUri}) for more information or tag **<b>@</b>dotnet/prodconsvcs** for assistance.";
private const string HelpLine = $"> 💡 You may consult the [FAQ]({PullRequestBuilder.CodeFlowPrFaqUri}) for more information or tag `` `@dotnet/prodconsvcs` `` for assistance.";

Copilot uses AI. Check for mistakes.
@premun premun merged commit ebe04bd into dotnet:main Nov 10, 2025
13 of 15 checks passed
@premun premun deleted the prvysoky/b branch November 10, 2025 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants