Skip to content
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

use xcode 12.0 #189

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

liulianci
Copy link
Contributor

No description provided.

@@ -24,6 +24,7 @@ jobs:
- uses: actions/setup-node@v1
with:
node-version: '12'
xcode-version: '12.0'
Copy link
Member

@friederbluemle friederbluemle Jul 28, 2021

Choose a reason for hiding this comment

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

I assume you intend to use the setup-xcode action here? Right now it's part of setup-node.

What we probably need is this:

- uses: maxim-lobanov/setup-xcode@v1
  with:
    xcode-version: '12.0'

I've added it to the existing workflow update PR #186, please confirm if that looks good. Also please confirm that we do need 12.0 here, and why. Interestingly they don't list 12.0 (but 12.0.1) for macos-latest (which is currently macOS 10.15).

See list of available versions here: https://github.com/actions/virtual-environments/blob/main/images/macos/macos-10.15-Readme.md#xcode

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants