Skip to content

remove xiangyan from codeowners - #2698

Open
xiangyan99 wants to merge 2 commits into
mainfrom
remove-xiangyan-from-codeowners
Open

remove xiangyan from codeowners#2698
xiangyan99 wants to merge 2 commits into
mainfrom
remove-xiangyan-from-codeowners

Conversation

@xiangyan99

Copy link
Copy Markdown
Member

remove xiangyan from codeowners

Copilot AI review requested due to automatic review settings May 21, 2026 15:54
@xiangyan99
xiangyan99 requested review from a team as code owners May 21, 2026 15:54

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

Removes @xiangyan99 from .github/CODEOWNERS entries so that ownership/auto-review requests for the affected areas no longer include that user.

Changes:

  • Updated multiple ServiceOwners/PRLabel entries to remove @xiangyan99.
  • Kept remaining owners/team aliases intact (e.g., @microsoft/azure-mcp) for the same paths/labels.

Invoking Livetests

Copilot submitted PRs are not trustworthy by default. Users with write access to the repo need to validate the contents of this PR before leaving a comment with the text /azp run mcp - pullrequest - live. This will trigger the necessary livetest workflows to complete required validation.

Show a summary per file
File Description
.github/CODEOWNERS Removes @xiangyan99 from relevant ownership entries while preserving existing ownership structure.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@github-actions

Copy link
Copy Markdown
Contributor

Hi @xiangyan99. Thank you for your interest in helping to improve the Azure SDK experience and for your contribution. We've noticed that there hasn't been recent engagement on this pull request. If this is still an active work stream, please let us know by pushing some changes or leaving a comment. Otherwise, we'll close this out in 7 days.

@RickWinter

Copy link
Copy Markdown
Member

@copilot resolve the merge conflicts in this pull request

…codeowners

# Conflicts:
#	.github/CODEOWNERS

Copilot AI commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

@copilot resolve the merge conflicts in this pull request

Merge conflicts have been resolved in commit d6b197d. The three conflicts in .github/CODEOWNERS were:

  1. Authorization PRLabel — merged by keeping @vurhanau @microsoft/azure-mcp (removing @xiangyan99 per this PR's goal, and not re-adding @jongio which main had already removed)
  2. Storage PRLabel — same approach: @alzimmermsft @microsoft/azure-mcp
  3. Duplicate Authorization section — removed entirely (main had cleaned this up)

Copilot AI requested a review from RickWinter July 29, 2026 06:16
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.

4 participants