Skip to content

Make webpack 4 compatible #98

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

Merged
merged 1 commit into from
Mar 3, 2018
Merged

Make webpack 4 compatible #98

merged 1 commit into from
Mar 3, 2018

Conversation

macisi
Copy link
Contributor

@macisi macisi commented Mar 2, 2018

Now the plugin works fine with webpack@4.0.0 and html-webpack-plugin@3.0.4. 😃

@SimenB SimenB changed the base branch from master to next March 3, 2018 07:19
@SimenB SimenB merged commit 2911f51 into SimenB:next Mar 3, 2018
@SimenB
Copy link
Owner

SimenB commented Mar 3, 2018

Thanks! I'll make some more tweaks before releasing 3.0.0 🙂

@JounQin
Copy link
Contributor

JounQin commented Mar 6, 2018

Error: Plugin could not be registered at 'html-webpack-plugin-before-html-generation'. on webpack 4.1.0, html-webpack-plugin 3.0.6

OK, I found out AddAssetHtmlPlugin should be added after HtmlWebpackPlugin for webpack@4+, and it is better to mention it in README.

@SimenB
Copy link
Owner

SimenB commented Mar 6, 2018

Oh, I didn't know that! Makes sense though. Mind sending a PR specifying that in the readme? 🙂

@JounQin
Copy link
Contributor

JounQin commented Mar 9, 2018

@SimenB Please review #103 .

SimenB pushed a commit that referenced this pull request Sep 15, 2018
SimenB pushed a commit that referenced this pull request Sep 15, 2018
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.

3 participants