Fix pipeline matrix job titles - #10448
Conversation
Remove the runtime matrix variable from the job display name so Azure DevOps renders the matrix legs as Windows Release and Windows Debug. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 0da639e0-4d35-424e-97e5-a063f615892c
There was a problem hiding this comment.
Pull request overview
Fixes Azure Pipelines matrix job titles so configuration names render correctly.
Changes:
- Removes the unresolved
_BuildConfigvariable from the Windows job display name. - Allows matrix leg names (
ReleaseandDebug) to be appended automatically.
Show a summary per file
| File | Description |
|---|---|
azure-pipelines.yml |
Simplifies the Windows matrix job display name. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Balanced
There was a problem hiding this comment.
Caution
agentic threat detected
Threat detection flagged this output in warn mode. Manual review is REQUIRED before any follow-up automation.
Details
Potential security threats were detected in the agent output.
Review the workflow run logs for details.
Note
🤖 Automated review by GitHub Copilot. Generated by the Expert Code Review workflow. To request a follow-up action, reply by tagging @copilot directly.
Expert Review — PR #10448
Scope: Removes the $(_BuildConfig) variable interpolation from the displayName of the Windows job in azure-pipelines.yml.
Verdict Table
| # | Dimension | Verdict |
|---|---|---|
| 1 | Algorithmic Correctness | N/A |
| 2 | Threading & Concurrency | N/A |
| 3 | Security & IPC Contract Safety | N/A |
| 4 | Public API & Binary Compatibility | N/A |
| 5 | Performance & Allocations | N/A |
| 6 | Cross-TFM Compatibility | N/A |
| 7 | Resource Management | N/A |
| 8 | Error Handling | N/A |
| 9 | Localization | N/A |
| 10 | Test Quality | N/A |
| 11 | Naming & Readability | ✅ Clean |
| 12 | Scope Discipline | ✅ Clean |
| 13 | Documentation | N/A |
| 14 | Logging & Diagnostics | N/A |
| 15 | Configuration & Build | ✅ Clean |
| 16 | Dependency Management | N/A |
| 17 | Telemetry | N/A |
| 18 | Extensibility | N/A |
| 19 | Serialization | N/A |
| 20 | Cancellation | N/A |
| 21 | Nullability | N/A |
| 22 | PowerShell Script Hygiene | N/A |
Summary
Single-line, minimal-risk change to CI configuration. The variable $(_BuildConfig) was redundant in this display name context (the stage/pipeline context already conveys the configuration). No issues found across all applicable dimensions.
Generated by Expert Code Review (on PR ready) for #10448 · auto · 65 AIC · ⌖ 0.745 AIC · ⊞ 10.4K
Summary
ReleaseandDebugmatrix leg namesWindows $(_BuildConfig) Release/DebugRelated build: https://dev.azure.com/dnceng-public/public/_build/results?buildId=1540570&view=logs&j=d89d42a5-6233-54f2-c754-4640cafeacb4