Skip to content

Fix: Updates compiler api#1

Open
smilebot wants to merge 1 commit into
mcjfunk:masterfrom
smilebot:webpack5-update
Open

Fix: Updates compiler api#1
smilebot wants to merge 1 commit into
mcjfunk:masterfrom
smilebot:webpack5-update

Conversation

@smilebot
Copy link
Copy Markdown

@smilebot smilebot commented Jan 4, 2021

compiler.plugin api has been deprecated. This change updates the code to use the newer hooks api

Copy link
Copy Markdown
Owner

@mcjfunk mcjfunk left a comment

Choose a reason for hiding this comment

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

This looks good. Did the API change with webpack 5 or was it earlier but deprecated? Just wondering what versions of webpack are supported here. If only webpack 5, please add a note to the README.

BTW, we might want to drop this from the scaffoldings and use one of the other solutions that are popular.

@smilebot
Copy link
Copy Markdown
Author

smilebot commented Jan 4, 2021

Hey Jeff, it was deprecated at an earlier version, I'm trying to track down when. Ex: this post from 2018 . This was breaking when we were trying to upgrade to webpack 5 from 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.

2 participants