[release/8.0-staging] Bump iOS & tvOS queues #123768
Conversation
All MacOS 13 queues no longer exist.
github-actions
bot
added
the
needs-area-label
| # tvOS devices | ||
| - ${{ if in(parameters.platform, 'tvos_arm64') }}: | ||
| - OSX.13.Amd64.AppleTV.Open | ||
|
|
There was a problem hiding this comment.
I'm not a YML-master... is this (now deleted) whitespace line important, or no?
There was a problem hiding this comment.
nope, indentation does. Blank lines don't
There was a problem hiding this comment.
Pull request overview
This pull request updates the Helix queue configurations for iOS and tvOS device testing from macOS 13 to macOS 15, as the macOS 13 queues are no longer available.
Changes:
- Updates iOS device queue from
OSX.13.Amd64.Iphone.OpentoOSX.15.Amd64.Iphone.Open - Updates tvOS device queue from
OSX.13.Amd64.AppleTV.OpentoOSX.15.Amd64.AppleTV.Open
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| eng/pipelines/libraries/helix-queues-setup.yml | Updates iOS and tvOS device queue configurations to use OSX.15 instead of OSX.13 for library tests |
| eng/pipelines/coreclr/templates/helix-queues-setup.yml | Updates iOS and tvOS device queue configurations to use OSX.15 instead of OSX.13 for CoreCLR tests |
| - ${{ if in(parameters.platform, 'tvos_arm64') }}: | ||
| - OSX.13.Amd64.AppleTV.Open | ||
|
|
||
| - OSX.15.Amd64.AppleTV.Open |
There was a problem hiding this comment.
The indentation of this line is inconsistent with the rest of the file. This line has 8 spaces of indentation, but it should have 6 spaces to match the iOS devices entry on line 59 and other queue entries in this file. This inconsistent indentation could cause YAML parsing issues or make the configuration harder to maintain.
| - OSX.15.Amd64.AppleTV.Open | |
| - OSX.15.Amd64.AppleTV.Open |
4 participants
All MacOS 13 queues no longer exist.