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

Update to Detox 9 #21570

Closed
wants to merge 1 commit into from
Closed

Update to Detox 9 #21570

wants to merge 1 commit into from

Conversation

kelset
Copy link
Contributor

@kelset kelset commented Oct 8, 2018

Updating to Detox 9 - this should help with better Xcode 10 support, and should fix #21539.

I've updated to fixed 9.0.4 since it seems that each version is fairly different from the next, and I've followed the migration guide for the android native dependencies.

Test Plan:

Run against CI. Should not affect other builds.

Release Notes:

[INTERNAL] [ENHANCEMENT] [DETOX] - Update to version 9.0.4

@kelset kelset added the 🔩Test Infrastructure Test infrastructure and continuous integration. label Oct 8, 2018
@kelset kelset requested a review from hramos as a code owner October 8, 2018 15:41
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 8, 2018
Copy link
Contributor

@facebook-github-bot facebook-github-bot left a comment

Choose a reason for hiding this comment

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

hramos has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.

@kelset kelset mentioned this pull request Oct 9, 2018
@hramos
Copy link
Contributor

hramos commented Oct 9, 2018

This should be landing soon.

@react-native-bot
Copy link
Collaborator

@kelset merged commit 15c0598 into facebook:master.

@facebook facebook locked as resolved and limited conversation to collaborators Oct 10, 2018
@react-native-bot react-native-bot added the Merged This PR has been merged. label Oct 10, 2018
grabbou pushed a commit that referenced this pull request Oct 11, 2018
Summary:
Updating to Detox 9 - this should help with better Xcode 10 support, and should fix #21539.

I've updated to fixed `9.0.4` since it seems that each version is fairly different from the next, and I've followed the [migration guide](https://github.com/wix/detox/blob/master/docs/Guide.Migration.md#migrating-from-detox-8xx-to-9xx) for the android native dependencies.

Release Notes:

[INTERNAL] [ENHANCEMENT] [DETOX] - Update to version 9.0.4

Pull Request resolved: #21570

Reviewed By: TheSavior

Differential Revision: D10242373

Pulled By: RSNara

fbshipit-source-id: bd29d554e0972f16d96b9b39b86c7143941d5b7c
t-nanava pushed a commit to microsoft/react-native-macos that referenced this pull request Jun 17, 2019
Summary:
Updating to Detox 9 - this should help with better Xcode 10 support, and should fix facebook#21539.

I've updated to fixed `9.0.4` since it seems that each version is fairly different from the next, and I've followed the [migration guide](https://github.com/wix/detox/blob/master/docs/Guide.Migration.md#migrating-from-detox-8xx-to-9xx) for the android native dependencies.

Release Notes:

[INTERNAL] [ENHANCEMENT] [DETOX] - Update to version 9.0.4

Pull Request resolved: facebook#21570

Reviewed By: TheSavior

Differential Revision: D10242373

Pulled By: RSNara

fbshipit-source-id: bd29d554e0972f16d96b9b39b86c7143941d5b7c
@hramos hramos deleted the kelset-update-detox-to-9 branch February 25, 2020 19:35
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. 🔩Test Infrastructure Test infrastructure and continuous integration.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Installation fails when path contains spaces due to using an obsolete version of detox
4 participants