Skip to content
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

Update for webpack5 #21

Merged
merged 3 commits into from Jul 22, 2021
Merged

Update for webpack5 #21

merged 3 commits into from Jul 22, 2021

Conversation

christianopaets
Copy link
Contributor

This can fix errors with the newest version of html-webpack-plugin and webpack5

Copy link
Owner

@Epimodev Epimodev left a comment

Choose a reason for hiding this comment

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

@christianopaets thanks a lot for this pull request!
As it's been a long time I didn't use webpack, I didn't try yet if it works well. But I've already a few suggestions. If you disagree don't hesitate to tell me and explain why :-)

src/plugin.js Outdated Show resolved Hide resolved
src/plugin.js Show resolved Hide resolved
src/plugin.js Outdated Show resolved Hide resolved
src/plugin.js Outdated Show resolved Hide resolved
@christianopaets
Copy link
Contributor Author

Thank you for your answer. I've made some changes and tested with old version of webpack and html-webpack-plugin. Now it works fine)))

src/plugin.js Outdated Show resolved Hide resolved
Copy link
Owner

@Epimodev Epimodev left a comment

Choose a reason for hiding this comment

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

Perfect thanks a lot!

@Epimodev Epimodev merged commit f911577 into Epimodev:master Jul 22, 2021
@Epimodev
Copy link
Owner

@christianopaets I just published version 2.4.0 with your changes

@christianopaets
Copy link
Contributor Author

@Epimodev thanks a lot

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.

None yet

2 participants