-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[No QA] Bumping Xcode version to fix iOS tests failing on Travis #5970
Conversation
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.
Nice work Luke 💪
✋ This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release. |
Bumping Xcode version to fix iOS tests failing on Travis (cherry picked from commit 467d3b1)
🚀 Cherry-picked to staging by @roryabraham in version: 1.1.8-7 🚀
@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes. |
🚀 Deployed to production by @roryabraham in version: 1.1.8-9 🚀
|
🚀 Deployed to staging by @Beamanator in version: 1.1.8-10 🚀
|
🚀 Deployed to production by @roryabraham in version: 1.1.10-2 🚀
|
Details
iOS tests are failing, so we need to figure out how to fix em
One idea was to bump the XCode version found in e2e.yml
Slack discussion: https://expensify.slack.com/archives/C01GTK53T8Q/p1634754414008400
Evidence for bumping: https://github.com/actions/virtual-environments/blob/main/images/macos/macos-11-Readme.md#xcode
We'll see if the iOS tests pass here.
Edit: Looks like they did! I think we should be good to merge this.