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

Simplify installation instructions #18

Merged
merged 1 commit into from
Feb 28, 2017
Merged

Conversation

swrobel
Copy link
Contributor

@swrobel swrobel commented Feb 28, 2017

Changes:

  • webpacker has moved package.json (and related files) to the project root from vendor/
  • There is no need to edit package.json directly. I added a yarn command to take care of adding the webpacker-react module & install it.

Fixes #11

```
Then you need to update your `package.json` file to include the `webpacker-react` NPM module:

`./bin/yarn add webpacker-react@0.1.x`
Copy link
Owner

Choose a reason for hiding this comment

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

./bin/yarn add webpacker-react adds "webpacker-react": "^0.1.0", to the package.json, which is better here.

@renchap
Copy link
Owner

renchap commented Feb 28, 2017

Good catch, thanks! It does not fixes #11 completely as we still need to update the test & example apps for the new Webpacker, but at least the doc will be correct.

@renchap renchap merged commit fe8b2e3 into renchap:master Feb 28, 2017
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.

2 participants