Skip to content

Add Reviewer Teams#323

Open
dbreshears wants to merge 7 commits intomainfrom
dev/devinb/codeowners
Open

Add Reviewer Teams#323
dbreshears wants to merge 7 commits intomainfrom
dev/devinb/codeowners

Conversation

@dbreshears
Copy link
Copy Markdown
Member

Adding reviewer teams to the CodeOwners.

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

Updates repository ownership rules to route reviews for various plugin/skill areas to reviewer teams instead of (or in addition to) specific individuals.

Changes:

  • Replaces individual CODEOWNERS entries for dotnet-msbuild with a reviewer team.
  • Adds reviewer teams alongside existing owners for several dotnet, dotnet-upgrade, dotnet-diag, and dotnet-data skill/test paths.

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

Copilot AI review requested due to automatic review settings March 11, 2026 09:10
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.


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

Agent-Logs-Url: https://github.com/dotnet/skills/sessions/0f052a87-406b-4ec3-aa67-57c2d11c64b4

Co-authored-by: danmoseley <6385855+danmoseley@users.noreply.github.com>
Copilot AI requested a review from danmoseley March 28, 2026 05:34
Copilot AI review requested due to automatic review settings April 8, 2026 19:08
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

Updates repository ownership rules to include additional reviewer teams for several .NET skill areas, broadening the set of default reviewers for changes under those paths.

Changes:

  • Add @dotnet/skills-*-reviewers teams to CODEOWNERS entries for msbuild, csharp language, upgrade, diag, data, and maui areas.
  • Extend ownership for existing paths (plugins/tests/agentic-workflows) rather than reassigning them.
Show a summary per file
File Description
.github/CODEOWNERS Adds reviewer teams to multiple ownership entries; currently contains duplicated csharp-scripts entries that should be deduplicated.

Copilot's findings

Tip

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

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

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings April 9, 2026 21:14
@danmoseley
Copy link
Copy Markdown
Member

@dbreshears do you still want to merge this

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

Updates the repository’s CODEOWNERS configuration to include additional @dotnet/* reviewer teams for relevant plugin/test/workflow paths, improving routing of review requests to appropriate team groups.

Changes:

  • Added reviewer team owners for dotnet-msbuild plugin/test/workflow paths.
  • Added reviewer team owners for selected dotnet, dotnet-upgrade, dotnet-diag, dotnet-data, and dotnet-maui skill/test scopes.
  • Kept existing individual/team owners while expanding coverage with reviewer teams.
Show a summary per file
File Description
.github/CODEOWNERS Adds @dotnet/* reviewer teams to multiple existing ownership entries while preserving current owners.

Copilot's findings

Tip

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants