Skip to content

Conversation

jlopez994
Copy link

It is not necessary to use a callback when html data is replaced. In fact, it doesn't work because they removed that attribute. So the fix is simple, just remove that line.
Reference: https://github.com/jantimon/html-webpack-plugin/blob/master/index.js#L193

It is not necessary to use a callback when html data is replaced. In fact, it doesn't work because they removed that attribute. So the fix is simple, just remove that line.
Reference: https://github.com/jantimon/html-webpack-plugin/blob/master/index.js#L193
@darkestblue91
Copy link

I have tested your change and the plugin works as expected, hope this will be included as soon as posible, it's needed with Webpack 4.

@DeviousM
Copy link

Just in case I've forked it and published the fixed one under a different name:
html-string-replace-webpack-plugin-webpack-4

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