Skip to content

Remove duplicate @ prefix from issueAuthor in GitOps#11568

Merged
dipeshmsft merged 1 commit into
dotnet:mainfrom
akoeplinger:fix-issueauthor-mentions-20260406
Apr 28, 2026
Merged

Remove duplicate @ prefix from issueAuthor in GitOps#11568
dipeshmsft merged 1 commit into
dotnet:mainfrom
akoeplinger:fix-issueauthor-mentions-20260406

Conversation

@akoeplinger

@akoeplinger akoeplinger commented Apr 6, 2026

Copy link
Copy Markdown
Member

https://github.com/GitOps-microsoft/GitOps.PullRequestIssueManagement/pull/262 (internal Microsoft link) changed the ${issueAuthor} placeholder to include the @ character.

Remove the one we added so we don't duplicate it.

Microsoft Reviewers: Open in CodeFlow

https://github.com/GitOps-microsoft/GitOps.PullRequestIssueManagement/pull/262 (internal Microsoft link) changed the `${issueAuthor}` placeholder to include the `@` character.

Remove the one we added so we don't duplicate it.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@akoeplinger akoeplinger requested review from a team and Copilot April 6, 2026 18:40
@dotnet-policy-service dotnet-policy-service Bot added the PR metadata: Label to tag PRs, to facilitate with triage label Apr 6, 2026

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Updates the GitOps.PullRequestIssueManagement policy reply text to avoid tagging the author with a duplicated @@ after ${issueAuthor} began including the @ prefix upstream.

Changes:

  • Remove the hard-coded @ before ${issueAuthor} in the servicing-consider auto-reply message.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@dipeshmsft dipeshmsft left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@dipeshmsft dipeshmsft merged commit 25bb2c4 into dotnet:main Apr 28, 2026
13 checks passed
@akoeplinger akoeplinger deleted the fix-issueauthor-mentions-20260406 branch April 28, 2026 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR metadata: Label to tag PRs, to facilitate with triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants