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

[WIP] 0.6.0 update DRAFT WIP 1 #30

Closed
wants to merge 37 commits into from
Closed

Conversation

brody4hire
Copy link
Owner

@brody4hire brody4hire commented Jul 11, 2019

Tested on Android & iOS, with and without --view option, on React Native versions 0.57, 0.58, and 0.59. (Known issues #28 & #29 were already raised concerning RN 0.60).

TODO items:

timojaask and others added 30 commits May 8, 2019 15:30
By these changes, developer's library can read android plugin version and react-native version of root project
- remove ignored react-native init options
- factor const execCommand string into its own line
- add react-native init command info to logging
Co-authored-by: Christopher J. Brody <chris@brody.consulting>
Co-authored-by: Timo Jääskeläinen <timojaask@users.noreply.github.com>
for iOS & Android

(with one TODO comment restored for iOS)
Christopher J. Brody added 4 commits July 10, 2019 21:22
defaults to react-native@0.59 for now

(use exampleReactNativeVersion="react-native@latest" to generate
 example with the most current react-native release)
for the sake of consistency with React Native 0.59
@brody4hire brody4hire self-assigned this Jul 11, 2019
@SaeedZhiany
Copy link
Contributor

SaeedZhiany commented Jul 11, 2019

you added some changes on my PR (the added backslashes) but you didn't change the android gradle version to 3.3.2 as you said in the quote. Although I really didn't think it is necessary, the newer versions of react-native (+0.59) are using version 3.4.1 and I believe it's better to keep it as it is.

@brody4hire brody4hire force-pushed the 0.6.0-update-dev-wip1 branch from ae0231d to b89063d Compare July 11, 2019 23:14
@brody4hire brody4hire force-pushed the 0.6.0-update-dev-wip1 branch from b89063d to 6d9bf13 Compare July 11, 2019 23:17
@brody4hire brody4hire closed this Jul 16, 2019
@brody4hire brody4hire mentioned this pull request Jul 16, 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.

3 participants