Skip to content

Remove benbp from CODEOWNERS - #63020

Merged
vcolin7 merged 2 commits into
mainfrom
RemovebenbpFromCodeowners
Sep 15, 2026
Merged

vcolin7 merged 2 commits into
mainfrom
RemovebenbpFromCodeowners

Conversation

@alzimmermsft

Copy link
Copy Markdown
Member

Remove benbp from CODEOWNERS

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 2 pipeline(s).
11 pipeline(s) were filtered out due to trigger conditions.
There may be pipelines that require an authorized user to comment /azp run to run.

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.

🔵 Needs a closer look

Apply the ownership change through the azsdk CODEOWNERS operation, then regenerate the rendered cache.

Pull request overview

Removes benbp ownership references from .github/CODEOWNERS.

Changes:

  • Updates /eng/ and docindex.yml ownership.
  • Removes the dedicated .config/1espt/ owner entry.
File summaries
File Summary
.github/CODEOWNERS Updates engineering and configuration ownership mappings.
Review details

Suppressed comments (1)

.github/CODEOWNERS:922

  • The owners workflow documents .github/CODEOWNERS as a rendered data-model artifact and says not to edit it directly. Removing these path owners only in this file can be overwritten and will not update the ownership cache; remove benbp through the azsdk CODEOWNERS operation for the corresponding associations, then regenerate/update the cache instead.
/eng/                                                              @danieljurek @raych1
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite (auto)

Note

Copilot is running an experiment and ran this review at Lite.


💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

@jsquire Jesse Squire (jsquire) 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.

jeez, Alan Zimmer (@alzimmermsft), his seat isn't even cold yet!

Comment thread .github/CODEOWNERS Outdated
@vcolin7
vcolin7 merged commit 7ef775f into main Sep 15, 2026
15 checks passed
@vcolin7
vcolin7 deleted the RemovebenbpFromCodeowners branch September 15, 2026 17:56
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