Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Chore: Upgrade Detox to 16.7.2 (#30084)
Summary: With Xcode 12 being the latest, Detox 15.x has issues - in particular, it means that if you try to `yarn install` the dependencies for the repo, you'll be greeted by this error: ```bash error /<stuff>/react-native/node_modules/detox: Command failed. Exit code: 1 Command: node scripts/postinstall.js Arguments: Directory: /<stuff>/react-native/node_modules/detox Output: /<stuff>/Library/Detox/ios/5824c837515589f21c08f09b716a6eda088aa31f was found, but could not find Detox.framework inside it. This means that the Detox framework build process was interrupted. deleting /<stuff>/Library/Detox/ios/5824c837515589f21c08f09b716a6eda088aa31f and trying to rebuild. Extracting Detox sources... Building Detox.framework from /<stuff>/Developer/OSS/react-native/node_modules/detox/ios_src... child_process.js:637 throw err; ^ ``` With the {emoji:1f44d} of hramos & alloy I've prep'd up a small defensive PR that can be quickly merged before cutting 0.64, that bumps the version of Detox from 15.4.4 to the highest version available within the reach of "no breaking changes" in changelog. The main reason why with 16.x this error doesn't happen is that from [16.0.0](https://github.com/wix/Detox/releases/tag/16.0.0): > Detox now comes as a prebuilt framework on iOS, thus lowering npm install times and saving some build issues that happen due to unexpected Xcode setups. It would have been better to update directly to latest (at the time of writing 17.7.1) but there are at least two versions that had changelogs that seem to involve bigger changes: * https://github.com/wix/Detox/releases/tag/17.4.7 * https://github.com/wix/Detox/releases/tag/16.8.0 Hopefully CI will will show that the bump doesn't break any test � ## Changelog <!-- Help reviewers and the release process by writing your own changelog entry. For an example, see: https://github.com/facebook/react-native/wiki/Changelog --> [Internal] [Changed] - Bumped Detox in the repo to 16.7.2 for Xcode 12 compatibility Pull Request resolved: #30084 Test Plan: Running yarn in the main repo with Node 14 & Xcode 12, without this change, will cause the error copy-pasted above. After upgrading to this version, the error disappear. Reviewed By: cpojer Differential Revision: D24293226 Pulled By: hramos fbshipit-source-id: 2b4d23b033be621274966262ec19200bee44df58
- Loading branch information