-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Another hot module reloading attempt #192
Conversation
}, | ||
"engines": { | ||
"node": ">=0.10.32" | ||
"node": ">=2.2.1" |
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.
node 2.2.1 ?
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.
Hmm, I thought one of the new dependencies required this version :/ My bad, I've force pushed a fix.
bcb0f61
to
a88045a
Compare
Hot loading remains in a separate process to allow server reloading Also unifies babel configs in .babelrc
a88045a
to
91cfa1e
Compare
Works for me! |
Excellent work, @stevoland!! 👍 🍻 |
cheers! 🎉 |
FYI when you'll be upgrading to react-transform-hmr there will be an error in the console |
* createApp method instead of app in app.js (erikras#192)
Another attempt at #157 but keeps the dev server in it's own process to keep server reloading as per glenjamin's suggestion: #157 (comment).
I think this PR also consolidates babel configs in
.babelrc
as per #53