Skip to content

Conversation

@arnabsen
Copy link
Contributor

react-native android is not a valid command. react-native eject will recreate the missing native folder.

Test Plan

  1. Delete android folder in project root
  2. In console enter "react-native run-android"
  3. Error message will now give hint to rebuild missing android folder via "react-native eject" command rather than non-existent "react-native android" command

Release Notes

[CLI] [BUGFIX] [local-cli/runAndroid/runAndroid.js] - Correct console log message to react-native eject

arnabsen and others added 8 commits January 19, 2019 00:14
* feat: build before publishing

* fix(tests): use jest.doMock instead of setMock

* chore: rename local-cli -> cli
* feat: build before publishing

* fix(tests): use jest.doMock instead of setMock

* chore: rename local-cli -> cli

* chore: get rid of local-cli leftovers

* fix: typo
* chore: setup monorepo with lerna

* chore: move flow to root

* fix: lint

* chore: add watch command
Add simple logger.
Introduce logger to bundle core and eject
@thymikee thymikee requested a review from grabbou January 23, 2019 18:05
@grabbou
Copy link
Member

grabbou commented Jan 23, 2019

Thank you! I am going to merge it now!

I decided to slightly change your PR to unify the command with iOS messages. Asking to run "eject" is an assumption - in most cases people are aware that they are working with an Expo project (which is not a React Native project per se).

@grabbou
Copy link
Member

grabbou commented Jan 23, 2019

Would you mind rebasing your PR? There were some changes (files moved around) in the meantime which makes it hard to rebase.

@arnabsen arnabsen changed the title Correct console log message to react-native eject Correct console log message for react-native android Jan 23, 2019
@arnabsen arnabsen closed this Jan 23, 2019
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.

5 participants