Skip to content

ci(iOS): Fix yarn install failing with Xcode 12 #460

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

Merged
merged 1 commit into from
Oct 22, 2020

Conversation

tido64
Copy link
Member

@tido64 tido64 commented Oct 14, 2020

Summary:

yarn install fails because Detox fails to build with Xcode 12.

Resolves #454.

Test Plan:

  1. Make sure the repo is clean: git clean -dfqx
  2. Run yarn

@tido64 tido64 requested a review from krizzu October 14, 2020 11:49
@tido64 tido64 linked an issue Oct 14, 2020 that may be closed by this pull request
@tido64 tido64 force-pushed the tido/fix-xcode12-build branch 3 times, most recently from d43a4c8 to abf5ade Compare October 15, 2020 20:59
@tido64 tido64 force-pushed the tido/fix-xcode12-build branch 5 times, most recently from e17c182 to f64c945 Compare October 21, 2020 21:36
@tido64 tido64 marked this pull request as ready for review October 21, 2020 21:50
@tido64 tido64 changed the title ci: Fix builds failing with Xcode 12 ci(iOS): Fix yarn install failing with Xcode 12 Oct 21, 2020
@tido64 tido64 force-pushed the tido/fix-xcode12-build branch from f64c945 to ea6ed78 Compare October 22, 2020 10:32
Note that this does not fix tests failing to run on iOS 14. EarlGrey,
which is what Detox depends on, crashes when run on iOS 14
(google/EarlGrey#1447). We may be required to
bump to latest Detox to solve this issue, but it will require more work
on our side.
@tido64 tido64 force-pushed the tido/fix-xcode12-build branch from ea6ed78 to cf8b9b9 Compare October 22, 2020 11:19
Copy link
Member

@krizzu krizzu left a comment

Choose a reason for hiding this comment

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

awesome 💪

@krizzu krizzu merged commit 1931c65 into master Oct 22, 2020
@tido64 tido64 deleted the tido/fix-xcode12-build branch October 22, 2020 16:38
@krizzu
Copy link
Member

krizzu commented Oct 24, 2020

🎉 This PR is included in version 1.13.1 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

Detox 12 not compatible with XCode 12, please upgrade detox
2 participants