Repository navigation
{Misc} Remove individual reviewers for PR - #10504
Conversation
|
Yu Chen (@jsntcy) One scope-consistency finding remains; this is a non-approving comment. Scope consistencyKeep the reviewer removal durable — Update the generator to preserve squad labels without recreating component-squad review requests under the described Scrum Master model. If only individual removal is intended, retain team requests instead. Verify with the changed Machine Learning rule and a fixed squad mapping that transformation preserves labels, adds no removed team requests, and is idempotent. This verification was not run. Upstream CI27 checks completed; 26 passed, 0 failed, 0 pending. No upstream failures to classify. These are upstream results, not Agent test execution. Test validation
⚖️ Risk assessment — 2/100 · LowThe Low rating reflects the detected change scope with no elevated security, reliability, customer, operational, dependency, sovereign-cloud, generated-output, or cross-component signal.
|
There was a problem hiding this comment.
🟡 Changes recommended
The weekly squad-mapping generator remains inconsistent with the policy and can propose restoring removed team review requests.
1 open issue
What changed in this PR
Updates PR automation for the squad-based Scrum Master model by removing many hard-coded reviewer and assignee actions.
Changes:
- Removes individual and squad review requests across service-specific rules.
- Removes selected assignee actions and personal mentions from review reminders.
- Preserves most service and squad labels.
| File | Description |
|---|---|
.github/policies/resourceManagement.yml |
Removes reviewer routing and simplifies selected review notifications. |
Review effort: Balanced
| - assignTo: | ||
| users: | ||
| - wangzelin007 | ||
| description: '[machine learning] Auto assign labels and reviewers based on PR title/description.' |
|
Misc |

🤖 PR Validation — ️✔️ All clear
With the transition to a squad-based Scrum Master model, assigning specific engineers/squads as reviewers is no longer required and we just assign the squad acting as scrum master as the reviewer.