Skip to content

Initial setup for webpack-4 #466

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

Closed
wants to merge 1 commit into from

Conversation

GeertWille
Copy link

Hey,

Initial setup for webpack 4. I tested some examples and they all seemed to work fine. Please test it out with your projects and post the issues you get here.

Thanks!

@denis-sokolov
Copy link
Collaborator

denis-sokolov commented Mar 29, 2018

  • A major version number bump. React-static exposes webpack config to the client, and in my case, I rely on Webpack 3 plugins. The upgrade will be breaking backwards compatibility.
  • extractCssChunks option relies on the extract-css-chunks-webpack-plugin plugin, which is not updated for Webpack 4.
  • Once we are able to migrate to Webpack 4, let’s remember to revisit webpack-serve in Consider webpack-serve over webpack-dev-server #503.

@@ -61,12 +59,12 @@
"extract-css-chunks-webpack-plugin": "^2.0.18",
"extract-hoc": "^0.0.5",
"extract-hoc-compose": "^0.0.1",
"extract-text-webpack-plugin": "^3.0.1",
"extract-text-webpack-plugin": "^4.0.0-beta.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

I believe this is no longer being supported or contributed to. See webpack-contrib/extract-text-webpack-plugin#749.

You may want to follow the pattern in this PR which uses mini-css-extract-plugin instead: https://github.com/phoenixframework/phoenix/pull/2831/files

@sudarshang
Copy link
Contributor

How can I help get this PR merged?

@JustFly1984
Copy link

JustFly1984 commented May 7, 2018 via email

@sudarshang
Copy link
Contributor

I have a new PR out #605, that addresses most of the concerns raised here. Please have a look and give feedback.

@tannerlinsley
Copy link
Contributor

Going with #605

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.

6 participants