Conversation
|
Azure Pipelines: Successfully started running 3 pipeline(s). 13 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
|
cc @steveisok |
|
Tagging subscribers to this area: @dotnet/runtime-infrastructure |
|
Tagging subscribers to 'arch-android': @vitek-karas, @simonrozsival, @steveisok, @akoeplinger |
There was a problem hiding this comment.
🟢 Approval recommended
No unresolved review issues were identified.
Pull request overview
Removes unsupported mobile queues from the .NET 8 extra-platforms CI pipeline to reduce servicing-build noise.
Changes:
- Removes Android, iOS/tvOS, simulator, and Mac Catalyst templates.
- Retains WASM, Linux Bionic, and other non-mobile jobs.
File summaries
| File | Description |
|---|---|
eng/pipelines/runtime-extra-platforms.yml |
Removes mobile template inclusions from the aggregate pipeline. |
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0
- Review effort level: Lite
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
|
@matouskozak FYI - the idea is basically that .NET 8 on mobile is already out of support, so it's unlikely we would fix anything we find in the CI anyway... |
|
should we remove the android as well @vitek-karas ? |
|
you mean Linux Bionic? I'd say yes. That said, .NET 8/9 go out of support in ~1.5months so not sure it matters much |
There was a problem hiding this comment.
🟡 Changes recommended
Restore the Linux Bionic template while removing the five mobile templates.
Get a fresh assessment by requesting another Copilot review.
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 1
- Review effort level: Lite
Replaces #134037
Closes #133998 (reduces CI noise)