Skip to content

Use macos12 machines in official build - #88793

Merged
agocke merged 5 commits into
dotnet:mainfrom
agocke:use-macos-12-on-official
Jul 13, 2023
Merged

Use macos12 machines in official build#88793
agocke merged 5 commits into
dotnet:mainfrom
agocke:use-macos-12-on-official

Conversation

@agocke

Copy link
Copy Markdown
Member

Scout out macos 12 images in official build only.

Scout out macos 12 images in official build only.
@ghostghost added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Jul 12, 2023
@ghostghost assigned agockeJul 12, 2023
Temporarily use macos12 in PR to check that everything works.
@agocke

Copy link
Copy Markdown
MemberAuthor

Also flighting this in PR first.

${{ if and(in(parameters.osGroup, 'osx', 'maccatalyst', 'ios', 'iossimulator', 'tvos', 'tvossimulator'), eq(variables['System.TeamProject'], 'public')) }}:
vmImage: 'macos-11'

${{ if and(in(parameters.osGroup, 'osx', 'maccatalyst', 'ios', 'iossimulator', 'tvos', 'tvossimulator'), ne(variables['System.TeamProject'], 'public')) }}:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you can use ${{ else }} here

Copy link
Copy Markdown
MemberAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wait, is that new? That has to be new.

@agocke

Copy link
Copy Markdown
MemberAuthor

Official build passed, merging.

@agocke
agocke merged commit 33ba157 into dotnet:mainJul 13, 2023
@agocke
agocke deleted the use-macos-12-on-official branch July 13, 2023 05:41
@ghostghost locked as resolved and limited conversation to collaborators Aug 13, 2023
Sign up for freeto subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

needs-area-labelAn area label is needed to ensure this gets routed to the appropriate area owners

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@agocke@jkoritzinsky@hoyosjs