-
Notifications
You must be signed in to change notification settings - Fork 5.3k
[release/10.0] Fix internal validation pipeline #123825
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: release/10.0
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -85,9 +85,11 @@ jobs: | |
| - OSX.15.Amd64.Open | ||
|
|
||
| # Android | ||
| - ${{ if in(parameters.platform, 'android_x86', 'android_x64', 'linux_bionic_x64') }}: | ||
| # Always use the Ubuntu-based Android queue for internal validation as there is no internal equivalent of | ||
| # the Windows.11.Amd64.Android.Open queue. | ||
| - ${{ if or(eq(variables['System.TeamProject'], 'internal'), in(parameters.platform, 'android_x86', 'android_x64', 'linux_bionic_x64')) }}: | ||
| - Ubuntu.2204.Amd64.Android.29.Open | ||
|
Member
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Why are we using open queues in internal?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We strip the .Open name in the libraries infra, so I wanted to keep consistent (Open queues don't actually work from internal branches). Technically I think we can drop the .Open here. |
||
| - ${{ if in(parameters.platform, 'android_arm', 'android_arm64', 'linux_bionic_arm', 'linux_bionic_arm64') }}: | ||
| - ${{ if and(eq(variables['System.TeamProject'], 'public'), in(parameters.platform, 'android_arm', 'android_arm64', 'linux_bionic_arm', 'linux_bionic_arm64')) }}: | ||
| - Windows.11.Amd64.Android.Open | ||
|
|
||
| # iOS Simulator/Mac Catalyst arm64 | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Inconsistent indentation on this line. The list item should be indented with 8 spaces to align with other queue entries in the file (see lines 73, 75, 82, etc.), but this line has 10 spaces.