-
-
Notifications
You must be signed in to change notification settings - Fork 632
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
Helpful Message After Generator Install #210
Helpful Message After Generator Install #210
Conversation
@aaronvb This looks awesome. Could you change "node-server-hot" to "express-server". I think that's a better name for the webpack dev server. @alexfedoseev @robwise Agree? Since there's no "cold", non-hot-reloading version of express, I think just |
|
793bd31
to
cb1fa73
Compare
@@ -10,7 +10,7 @@ | |||
"scripts": { | |||
"postinstall": "cd client && npm install", | |||
"test": "rspec && (cd client && npm run lint)", | |||
"node-server-hot": "echo 'visit http://localhost:4000' && cd client && npm start", | |||
"express-server": "echo 'visit http://localhost:4000' && cd client && npm start", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the npm run command node-server-hot
to express-server
.
- Added a helpful message after base generator install. - Added a helpful message after react with redux generator install. - Added tests for both messages. - Change 'npm run node-server-hot' to 'npm run express-server'
cb1fa73
to
89dd8b4
Compare
Thanks @aaronvb. Is this ready for merge and a dot release? |
@justin808 I just pushed an edit to the |
Helpful Message After Generator Install
- Change text node-server-hot to express-server. This was changed in shakacode#210 - Fix indentation.
rails generate react_on_rails:install
rails generate react_on_rails:install -R
Resloves #184