Skip to content

fix(agents): cover system icons review routing - #36866

Merged
Martin Hochel (Hotell) merged 1 commit into
microsoft:masterfrom
Hotell:fix/assign-prs-system-icons-routing
Oct 12, 2026
Merged

Martin Hochel (Hotell) merged 1 commit into
microsoft:masterfrom
Hotell:fix/assign-prs-system-icons-routing

Conversation

@Hotell

Copy link
Copy Markdown
Contributor

Summary

  • Default unmatched system-icons paths to build/CI without extending the schema.
  • Define explicit runtime icon-package and documentation/demo exceptions; preserve assets/native-platform routing and hard area matching.
  • Add eslint-plugin-react-icons and webpack title scopes. Include docs/build ownership updates for existing eligible reviewers without changing eligibility flags.
  • Add regression coverage for all 11 current package directories, root configuration, undefined packages, ownership exceptions, and tooling scopes.

Validation

  • node --test .agents/skills/assign-prs/scripts/planner.test.mjs: 12 tests passed, including schema validation.
  • git diff --check: passed.

Notes

  • Mixed-area routing still uses file counts; newly undefined packages default to build.
  • No review requests were made by this run. No published packages changed; no beachball change file is required.

@Hotell
Martin Hochel (Hotell) requested a review from a team as a code owner October 12, 2026 10:07
@github-actions

Copy link
Copy Markdown

📊 Bundle size report

✅ No changes found

@github-actions

Copy link
Copy Markdown

Pull request demo site: URL

@Hotell
Martin Hochel (Hotell) merged commit 389e06c into microsoft:master Oct 12, 2026
13 checks passed
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.

2 participants