Sync area notification subscribers with area-owners.md by Copilot · Pull Request #122357 · dotnet/runtime

Copilot · 2025-12-09T19:11:39Z

Description

The resourceManagement.yml notification configuration had areas out of sync with area-owners.md. Notifications were being sent to incorrect or outdated subscribers, and area leads were not always included in notifications. Additionally, area-owners.md had incorrect or incomplete entries for several areas.

Updated .github/policies/resourceManagement.yml to sync all area label notification subscribers with docs/area-owners.md. The approach prioritizes using dotnet/* teams for automated notifications while ensuring individual experts are properly listed in the owners column of area-owners.md where they should receive direct notifications.

@agocke

Update resourceManagement.yml to match area-owners.md for area-AssemblyLoader-coreclr. The mentionees are now @agocke and @elinor-fung, matching the owners listed in area-owners.md. Co-authored-by: agocke <[email protected]>

.github/policies/resourceManagement.yml

Updated resourceManagement.yml to sync all area label notification subscribers with docs/area-owners.md. Key changes: - area-Host: vitek-karas, agocke, vsadov -> jeffschwMSFT, elinor-fung - area-HostModel: vitek-karas, agocke -> elinor-fung - area-AssemblyLoader-mono: [] -> agocke, elinor-fung - area-CodeGen-coreclr: JulieLeeMSFT, jakobbotsch -> dotnet/jit-contrib - area-GC-coreclr: dotnet/gc -> Maoni0 - area-Infrastructure: dotnet/runtime-infrastructure -> jeffschwMSFT, MichaelSimons - And 27 other areas synced with their documented owners Total: 33 areas updated to match area-owners.md Co-authored-by: agocke <[email protected]>

.github/policies/resourceManagement.yml

Updated resourceManagement.yml to include both the Lead and Owners columns from area-owners.md in notification subscribers, as requested. Previously only included Owners column, now includes Lead + Owners (with Lead first if not already in Owners list). Key examples: - area-Codegen-Intrinsics-mono: [] -> steveisok (Lead only, no owners) - area-Build-mono: akoeplinger -> lewing, akoeplinger (added Lead) - area-Host: jeffschwMSFT, elinor-fung -> agocke, jeffschwMSFT, elinor-fung (added Lead) - area-GC-coreclr: Maoni0 -> agocke, Maoni0 (added Lead) - And 92 other areas updated to include Lead Total: 96 areas updated to include both Lead and Owners Co-authored-by: agocke <[email protected]>

.github/policies/resourceManagement.yml

@Maoni0

Updated area-owners.md to correct the Owners for area-GC-coreclr from @Maoni0 to @dotnet/gc. Also updated resourceManagement.yml to match the corrected area-owners.md. This ensures the GC team is notified for area-GC-coreclr issues rather than just an individual. Co-authored-by: agocke <[email protected]>

For areas where jeffhandley is the lead and the owners include dotnet/* teams, removed jeffhandley from the notification subscribers since the team already covers the notification needs. This affects 59 areas including: - area-System.Diagnostics.EventLog: jeffhandley, dotnet/area-system-diagnostics-eventlog -> dotnet/area-system-diagnostics-eventlog - area-System.Buffers: jeffhandley, dotnet/area-system-buffers -> dotnet/area-system-buffers - area-Extensions-* (9 areas) - area-System.* (45 areas) - And other areas where jeffhandley leads teams The dotnet/* teams already include the necessary members for notifications. Co-authored-by: agocke <[email protected]>

agocke

@copilot Please address comments

.github/policies/resourceManagement.yml

@agocke

Per @agocke's feedback, added bartonjs and vcsjones to: 1. resourceManagement.yml mentionees for area-System.Security 2. area-owners.md owners column for area-System.Security (moved from consultants) This ensures they receive direct notifications for area-System.Security, consistent with how they're configured for area-System.Runtime.Intrinsics. Co-authored-by: agocke <[email protected]>

agocke · 2026-01-06T19:09:00Z

@SamMonoRT @phil-allen-msft @jeffhandley @karelz

I synced up the area notifications with area owners. These were out of date. Can you take a look and make sure this is correct for your team?

Copilot

Pull request overview

This PR synchronizes area label notification subscribers between .github/policies/resourceManagement.yml and docs/area-owners.md. The changes ensure that notifications are sent to the correct area leads and teams, with outdated individual subscribers being replaced by team references (e.g., dotnet/jit-contrib, dotnet/interop-contrib, dotnet/runtime-infrastructure) or updated to reflect current ownership assignments.

Key changes include:

Adding area leads to notification lists where they were missing (e.g., @agocke, @karelz, @phil-allen-msft)
Replacing individual subscribers with team references for better maintainability (e.g., dotnet/jit-contrib, dotnet/ilc-contrib, dotnet/interop-contrib)
Correcting GitHub handle casing for consistency (e.g., brzvlad → BrzVlad, SamMonoRT → sammonort)
Removing consultants from automatic notifications (they remain listed in area-owners.md for reference)

Reviewed changes

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

File	Description
docs/area-owners.md	Updated owner assignments to reflect current team structures; added individual crypto experts (@bartonjs, @vcsjones) to several security-related areas; corrected lead assignments for archived components and infrastructure areas
.github/policies/resourceManagement.yml	Synchronized notification subscribers with area-owners.md; added missing area leads to notification lists; replaced individual contributors with team references; corrected GitHub handle casing; removed outdated or consultant-only subscribers

docs/area-owners.md

agocke · 2026-01-12T23:18:10Z

@SamMonoRT @phil-allen-msft @jeffhandley @karelz Any more comments?

agocke · 2026-01-12T23:18:21Z

/ba-g docs-only change

jeffhandley

I see a couple changes to make. It'll be easiest for me to push to the branch, which I'll do shortly.

agocke · 2026-01-13T19:05:26Z

Also @steveisok

docs/area-owners.md

.github/policies/resourceManagement.yml

jeffhandley · 2026-01-14T01:05:05Z

/ba-g docs only

AndriySvyryd · 2026-01-14T20:22:31Z

docs/area-owners.md

+| area-System.Data                               | @sammonort           | @dotnet/efteam                                                                           | <ul><li>Odbc, OleDb - @saurabh500</li></ul>                                                                                                                                                                                                                                               |
+| area-System.Data.Odbc                          | @sammonort           | @dotnet/efteam                                                                           |                                                                                                                                                                                                                                                                                           |
+| area-System.Data.OleDB                         | @sammonort           | @dotnet/efteam                                                                           |                                                                                                                                                                                                                                                                                           |


AndriySvyryd · 2026-01-14T20:23:22Z

docs/area-owners.md

 | area-System.Threading.Channels                 | @jeffhandley         | @dotnet/area-system-threading-channels                                                   | Consultants: @stephentoub                                                                                                                                                                                                                                                                 |
 | area-System.Threading.RateLimiting             | @rafikiassumani-msft | @BrennanConroy @halter73                                                                 |                                                                                                                                                                                                                                                                                           |
 | area-System.Threading.Tasks                    | @jeffhandley         | @dotnet/area-system-threading-tasks                                                      | Consultants: @stephentoub                                                                                                                                                                                                                                                                 |
 | area-System.Transactions                       | @sammonort           | @roji @cincuranet                                                                        |                                                                                                                                                                                                                                                                                           |


@AndriySvyryd

Follow-up to [Sync area notification subscribers with area-owners.md (#122357)](#122357), capturing feedback from @AndriySvyryd that came in after merge.

EgorBo · 2026-02-02T18:41:24Z

.github/policies/resourceManagement.yml

            mentionees:
            - JulieLeeMSFT
-            - jakobbotsch
+            - dotnet/jit-contrib


@agocke area-codegen is one of the (actually the most) popular label in the repo, this PR subscribed our team to all codegen issues 😞

Revert the change done in #122357 `area-CodeGen-coreclr` label is by far [the most popular label](https://github.com/dotnet/runtime/labels?sort=count-desc) in dotnet/runtime. This creates a huge mailbox mess and we're more likely to miss direct pings. We usually iterate all issues with that label anyway.

Revert the change done in dotnet#122357 `area-CodeGen-coreclr` label is by far [the most popular label](https://github.com/dotnet/runtime/labels?sort=count-desc) in dotnet/runtime. This creates a huge mailbox mess and we're more likely to miss direct pings. We usually iterate all issues with that label anyway.

Initial plan

b4ad26a

Copilot AI assigned Copilot and agocke Dec 9, 2025

Copilot started work on behalf of agocke December 9, 2025 19:12 View session

Copilot AI changed the title ~~[WIP] Update area-AssemblyLoader-coreclr in resourceManagement file~~ Sync area-AssemblyLoader-coreclr notification subscribers with area-owners.md Dec 9, 2025

Copilot AI requested a review from agocke December 9, 2025 19:42

Copilot finished work on behalf of agocke December 9, 2025 19:42

agocke reviewed Dec 9, 2025

View reviewed changes

.github/policies/resourceManagement.yml Show resolved Hide resolved

Copilot started work on behalf of agocke December 9, 2025 21:33 View session

Copilot AI changed the title ~~Sync area-AssemblyLoader-coreclr notification subscribers with area-owners.md~~ Sync all area notification subscribers with area-owners.md Dec 9, 2025

Copilot AI requested a review from agocke December 9, 2025 21:59

Copilot finished work on behalf of agocke December 9, 2025 21:59

agocke reviewed Dec 9, 2025

View reviewed changes

.github/policies/resourceManagement.yml Show resolved Hide resolved

Copilot started work on behalf of agocke December 9, 2025 23:57 View session

Copilot AI changed the title ~~Sync all area notification subscribers with area-owners.md~~ Sync all area notification subscribers with area-owners.md (Lead + Owners) Dec 10, 2025

Copilot AI requested a review from agocke December 10, 2025 00:23

Copilot finished work on behalf of agocke December 10, 2025 00:23

agocke reviewed Dec 10, 2025

View reviewed changes

.github/policies/resourceManagement.yml Outdated Show resolved Hide resolved

Copilot started work on behalf of agocke December 10, 2025 02:18 View session

agocke reviewed Dec 10, 2025

View reviewed changes

.github/policies/resourceManagement.yml Outdated Show resolved Hide resolved

Copilot AI requested a review from agocke December 10, 2025 02:44

Copilot finished work on behalf of agocke December 10, 2025 02:44

Copilot started work on behalf of agocke December 10, 2025 02:45 View session

Copilot finished work on behalf of agocke December 10, 2025 03:09

agocke reviewed Dec 11, 2025

View reviewed changes

Copilot AI requested a review from agocke January 5, 2026 19:22

Copilot started work on behalf of agocke January 5, 2026 21:17 View session

Copilot finished work on behalf of agocke January 5, 2026 21:40

agocke marked this pull request as ready for review January 6, 2026 19:07

agocke requested a review from jeffhandley as a code owner January 6, 2026 19:07

Copilot AI review requested due to automatic review settings January 6, 2026 19:07

Copilot started reviewing on behalf of agocke January 6, 2026 19:08 View session

Copilot AI reviewed Jan 6, 2026

View reviewed changes

docs/area-owners.md Show resolved Hide resolved

jeffhandley reviewed Jan 13, 2026

View reviewed changes

Apply feedback for libraries mentionees

90a8ce1

jeffhandley reviewed Jan 14, 2026

View reviewed changes

docs/area-owners.md Outdated Show resolved Hide resolved

.github/policies/resourceManagement.yml Outdated Show resolved Hide resolved

jeffhandley approved these changes Jan 14, 2026

View reviewed changes

SamMonoRT approved these changes Jan 14, 2026

View reviewed changes

agocke merged commit 06235f4 into main Jan 14, 2026
16 of 17 checks passed

agocke deleted the copilot/sync-area-assemblyloader-coreclr branch January 14, 2026 19:48

AndriySvyryd reviewed Jan 14, 2026

View reviewed changes

jeffhandley mentioned this pull request Jan 14, 2026

Update remaining efteam ownership area. Fix capitalization. #123199

Merged

dotnet-maestro bot mentioned this pull request Jan 15, 2026

[main] Source code updates from dotnet/runtime dotnet/dotnet#4220

Merged

EgorBo reviewed Feb 2, 2026

View reviewed changes

EgorBo mentioned this pull request Feb 3, 2026

Don't ping codegen team for codegen label #123945

Merged


	\| area-System.Transactions \| @SamMonoRT \| @dotnet/efteam \| \|

Conversation

Copilot AI commented Dec 9, 2025 • edited by agocke Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Description

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

agocke left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

agocke commented Jan 6, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Uh oh!

agocke commented Jan 12, 2026

Uh oh!

agocke commented Jan 12, 2026

Uh oh!

jeffhandley left a comment

Choose a reason for hiding this comment

Uh oh!

agocke commented Jan 13, 2026

Uh oh!

Uh oh!

Uh oh!

jeffhandley commented Jan 14, 2026

Uh oh!

Uh oh!

AndriySvyryd Jan 14, 2026

Choose a reason for hiding this comment

Uh oh!

AndriySvyryd Jan 14, 2026

Choose a reason for hiding this comment

Uh oh!

EgorBo Feb 2, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

Copilot AI commented Dec 9, 2025 •

edited by agocke

Loading