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

Added react-native run-ios #5119

Closed
wants to merge 6 commits into from
Closed

Added react-native run-ios #5119

wants to merge 6 commits into from

Conversation

frantic
Copy link
Contributor

@frantic frantic commented Jan 4, 2016

Works the same way as react-native run-android, but targets iOS simulator instead. Under the hood, it uses xcodebuild to compile the app and store it in ios/build folder, then triggers instruments and simctl to install and launch the app on simulator.

Since Facebook relies on BUCK to build and run iOS app, we probably won't use run-ios internally. That's why I'm putting this as public PR instead of internal diff.

To test this, I hacked global react-native script to install react native from my local checkout instead of from npm, cd into the folder and ran react-native run-ios.

@facebook-github-bot
Copy link
Contributor

By analyzing the blame information on this pull request, we identified @martinbigio, @mkonicek and @frantic to be potential reviewers.

@facebook-github-bot facebook-github-bot added GH Review: review-needed CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. labels Jan 4, 2016
@facebook-github-bot
Copy link
Contributor

@frantic updated the pull request.

@skevy
Copy link
Contributor

skevy commented Jan 4, 2016

This is awesome! +1 to opening xcode as little as possible.

@martinbigio
Copy link
Contributor

This is awesome! @vjeux and I were talking about this a few days ago :)

* Parses the output of `xcrun simctl list devices` command
*/
function parseIOSSimulatorsList(text: string): Array<IOSSimulatorInfo> {
var devices = [];
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: const

@facebook-github-bot
Copy link
Contributor

@frantic updated the pull request.

@dsibiski
Copy link
Contributor

dsibiski commented Jan 5, 2016

This is great! 😍

@martinbigio
Copy link
Contributor

The implementation looks good to me, @frantic feel free to merge it.

@frantic
Copy link
Contributor Author

frantic commented Jan 5, 2016

@facebook-github-bot shipit

@facebook-github-bot
Copy link
Contributor

Thanks for importing. If you are an FB employee go to https://our.intern.facebook.com/intern/opensource/github/pull_request/1688959801386516/int_phab to review.

@frantic frantic closed this in 9490c2c Jan 6, 2016
christopherdro pushed a commit to wildlifela/react-native that referenced this pull request Jan 20, 2016
Summary:
Works the same way as `react-native run-android`, but targets iOS simulator instead. Under the hood, it uses `xcodebuild` to compile the app and store it in `ios/build` folder, then triggers `instruments` and `simctl` to install and launch the app on simulator.

Since Facebook relies on BUCK to build and run iOS app, we probably won't use `run-ios` internally. That's why I'm putting this as public PR instead of internal diff.

To test this, I hacked global `react-native` script to install react native from my local checkout instead of from npm, cd into the folder and ran `react-native run-ios`.
Closes facebook#5119

Reviewed By: svcscm

Differential Revision: D2805199

Pulled By: frantic

fb-gh-sync-id: 423a45ba885cb5e48a16ac22095d757d8cca7e37
grabbou pushed a commit to react-native-community/cli that referenced this pull request Sep 26, 2018
Summary:
Works the same way as `react-native run-android`, but targets iOS simulator instead. Under the hood, it uses `xcodebuild` to compile the app and store it in `ios/build` folder, then triggers `instruments` and `simctl` to install and launch the app on simulator.

Since Facebook relies on BUCK to build and run iOS app, we probably won't use `run-ios` internally. That's why I'm putting this as public PR instead of internal diff.

To test this, I hacked global `react-native` script to install react native from my local checkout instead of from npm, cd into the folder and ran `react-native run-ios`.
Closes facebook/react-native#5119

Reviewed By: svcscm

Differential Revision: D2805199

Pulled By: frantic

fb-gh-sync-id: 423a45ba885cb5e48a16ac22095d757d8cca7e37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants