Skip to content

Add Identity to CODEOWNERS - #69012

Open
BrennanConroy wants to merge 1 commit into
mainfrom
BrennanConroy-patch-1
Open

Add Identity to CODEOWNERS#69012
BrennanConroy wants to merge 1 commit into
mainfrom
BrennanConroy-patch-1

Conversation

@BrennanConroy

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings September 3, 2026 04:02
@BrennanConroy
BrennanConroy requested review from a team and wtgodbe as code owners September 3, 2026 04:02

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.

Copilot review overview

🟡 Changes recommended

The PR title suggests only adding Identity ownership, but it also changes PublicAPI shipped-file ownership and needs clarification or adjustment to avoid an unexpected reviewer-coverage change.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Lite
Findings: 1 Medium severity

New issues introduced by this change (1)
Severity Finding
Medium severity .github/​CODEOWNERS — This PR is titled "Add Identity to CODEOWNERS", but it also changes the owners for…
What changed in this PR

Updates the repository’s CODEOWNERS configuration to ensure Identity-area changes route to the right reviewers.

Changes:

  • Adds CODEOWNERS coverage for /src/Identity/.
  • Modifies the CODEOWNERS entry for /**/PublicAPI.*Shipped.txt by changing the listed owners.
File Description
.github/​CODEOWNERS Adds Identity owners and adjusts the owners list for shipped PublicAPI files.

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

Comment thread .github/CODEOWNERS
/src/SignalR/ @BrennanConroy @halter73
/src/submodules @dotnet/aspnet-build @wtgodbe
/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @Rick-Anderson @tdykstra
/**/PublicAPI.*Shipped.txt @dotnet/aspnet-api-review @tdykstra
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