Skip to content

Consolidate area labels: merge area-Single-File and area-HostModel into area-Host#126734

Open
Copilot wants to merge 4 commits intomainfrom
copilot/remove-rename-area-paths
Open

Consolidate area labels: merge area-Single-File and area-HostModel into area-Host#126734
Copilot wants to merge 4 commits intomainfrom
copilot/remove-rename-area-paths

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 9, 2026

Description

Consolidates overlapping area labels into area-Host:

  • area-Single-File → merged into area-Host; row removed, description and heuristics updated
  • area-HostModel → merged into area-Host; row removed, description updated
  • Removed @vsadov from area-Host contributors
  • Updated @dotnet-policy-service configuration to remove area-Single-File and area-HostModel label triggers and notification blocks

Changes

  • docs/area-owners.md: Removed area-Single-File and area-HostModel rows; updated area-Host description to cover single-file deployment and host model
  • .github/skills/issue-triage/references/area-label-heuristics.md: Merged single-file deployment heuristic into the area-Host line
  • .github/policies/resourceManagement.yml: Removed area-HostModel and area-Single-File labelAdded triggers and hasLabel/mentionUsers notification blocks so policy automation is in sync with the consolidated area-Host label

Copilot AI requested review from Copilot and removed request for Copilot April 9, 2026 23:19
@agocke agocke marked this pull request as ready for review April 9, 2026 23:20
@agocke agocke requested a review from jeffhandley as a code owner April 9, 2026 23:20
Copilot AI review requested due to automatic review settings April 9, 2026 23:20
@dotnet-policy-service
Copy link
Copy Markdown
Contributor

Tagging subscribers to this area: @agocke, @jeffschwMSFT, @elinor-fung
See info in area-owners.md if you want to be subscribed.

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

Consolidates area-label documentation and triage heuristics by merging area-Single-File and area-HostModel guidance into area-Host, so Host-related ownership and labeling guidance is centralized.

Changes:

  • Updates area-Host ownership description and removes area-Single-File / area-HostModel rows from the area owners table.
  • Updates issue-triage heuristics to route single-file deployment issues to area-Host.
  • Also changes Crossgen/R2R label guidance to use area-crossgen and removes the area-ReadyToRun-coreclr row.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 5 comments.

File Description
docs/area-owners.md Removes area-Single-File/area-HostModel entries and expands area-Host scope; also adjusts Crossgen/ReadyToRun-related entries.
.github/skills/issue-triage/references/area-label-heuristics.md Updates quick-reference heuristics to map single-file (and other terms) to area-Host, and updates Crossgen/R2R heuristic label.

| area-GC-mono | @agocke | @BrzVlad | @agocke to consult |
| area-Host | @agocke | @jeffschwMSFT @elinor-fung | Issues with dotnet.exe including bootstrapping, framework detection, hostfxr.dll and hostpolicy.dll |
| area-HostModel | @agocke | @elinor-fung | |
| area-Host | @agocke | @jeffschwMSFT @elinor-fung | Issues with dotnet.exe including bootstrapping, framework detection, hostfxr.dll, hostpolicy.dll, host model, and single-file deployment |
Copy link

Copilot AI Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

area-Single-File and area-HostModel are removed from this table, but @dotnet-policy-service configuration still references both labels (in .github/policies/resourceManagement.yml). If these labels are being consolidated into area-Host, the policy mapping/notifications should be updated accordingly; otherwise issues/PRs labeled with the old areas will still route separately and this table will be out of sync with automation.

Copilot uses AI. Check for mistakes.
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.

@copilot apply changes based on this feedback

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed both area-HostModel and area-Single-File from .github/policies/resourceManagement.yml — removed the labelAdded triggers and the hasLabel/mentionUsers notification blocks for both. The area-Host block remains with the correct mentionees (agocke, jeffschwMSFT, elinor-fung). See commit 357c8d7.

@agocke agocke changed the title Consolidate area labels: merge area-Single-File and area-HostModel into area-Host Consolidate area labels in hosting and crossgen Apr 10, 2026
Copilot AI changed the title Consolidate area labels in hosting and crossgen Consolidate area labels: merge area-Single-File and area-HostModel into area-Host Apr 10, 2026
Copilot AI requested a review from agocke April 10, 2026 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants