-
Notifications
You must be signed in to change notification settings - Fork 147
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
Webpack 4 #242
Conversation
package.json
Outdated
"fs-extra": "^5.0.0", | ||
"html-webpack-plugin": "^2.30.1", | ||
"html-webpack-plugin": "^3.0.7cd tem ", |
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.
@pruett thanks for pointing out! Fixed 👍 |
// if (process.env.NODE_ENV === 'production') { ... }. See `./env.js`. | ||
// It is absolutely essential that NODE_ENV was set to production here. | ||
// Otherwise React will be compiled in the very slow development mode. | ||
new webpack.DefinePlugin(env.stringified), |
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.
new webpack.DefinePlugin(env.stringified)
, is added twice
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.
Good catch, thanks!
since facebook/create-react-app#4077 is merged now. |
None of the dependencies were released to NPM, the progress can be tracked here facebook/create-react-app#3815 |
Create Elm App with Webpack 4 is available at Please let me know if you are experiencing any issues! |
…ing it on precommit):
… to support webpack@4.#
Upgraded Webpack loaders and fs-extra BREAKING CHANGE: react-dev-utils is incompatible with the old one, therefore it is containing a potential breaking change.
Remove the tests for Node 4 and 6 BREAKING CHANGE: Node 4 and 6 are no longer supported.
BREAKING CHANGE: Expected engine major version upgrade
…tting for vendor stuff.
This PR enables Webpack 4 support.
CLI
yarn.lock
Dev
Dependencies: facebook/create-react-app#4077
While it is possible to update the InterpolateHtmlPlugin and use a temporary replacement, I think it makes sense to wait for CRA release and release CEA with all the updates.
Closes #241