Skip to content

Fix Usage on README #89

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

Merged
merged 1 commit into from
Oct 10, 2019

Conversation

julioxavierr
Copy link
Contributor

@julioxavierr julioxavierr commented Oct 9, 2019

Summary

Apparently currently the option --template already adds the react-native-template to the path, and if we use what's on README we will have this path that doesn't exist:

https://registry.yarnpkg.com/react-native-template-react-native-template-typescript

Changing to just --template typescript fixes it.

Test Plan

What's required for testing (prerequisites)?

Follow the README Usage instructions

What are the steps to reproduce (after prerequisites)?

Compatibility

OS Implemented
iOS
Android

Checklist

  • I have tested this on a device and a simulator
  • I added the documentation in README.md
  • I mentioned this change in CHANGELOG.md
  • I updated the typed files (TS and Flow)
  • I added a sample use of the API in the example project (example/App.js)

Apparently currently the option `--template` already adds the `react-native-template` to the path, and if we use what's on README we will have this path that doesn't exist:

https://registry.yarnpkg.com/react-native-template-react-native-template-typescript

Changing to just `--template typescript` fixes it.
@MateusAndrade MateusAndrade merged commit 97560ed into react-native-community:master Oct 10, 2019
@emin93
Copy link
Member

emin93 commented Oct 10, 2019

This won't work with the new CLI, see here: #62.

@radko93
Copy link
Collaborator

radko93 commented Oct 10, 2019

As @emin93 noted this wont' work #62 (comment), I will submit a PR to revert it. You're using the old CI.

radko93 added a commit that referenced this pull request Oct 10, 2019
radko93 added a commit that referenced this pull request Oct 10, 2019
@julioxavierr julioxavierr deleted the patch-1 branch October 12, 2019 11:28
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