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

chore: initial work on webpack 5 integration #458

Closed
wants to merge 1 commit into from

Conversation

andriijas
Copy link

This PR contains a:

  • new feature

Motivation / Use-Case

Webpack 5 is coming, lets work together to make this plugin compatible!

Breaking Changes

Not sure, this PR should be considered DRAFT and not ready yet

Additional Info

Im a webpack amateur. But there is no better way to try to learn than practice, right?

Please provide feedback if this is on the right track and what more thinks to consider.

I would also like to know if there is a way to make the plugin compatible with both webpack 4 and 5.

Known issues

    ERROR in [initial]
    chunk.getChunkMaps is not a function TypeError: chunk.getChunkMaps is not a function

@jsf-clabot
Copy link

jsf-clabot commented Oct 31, 2019

CLA assistant check
All committers have signed the CLA.

@alexander-akait
Copy link
Member

alexander-akait commented Nov 5, 2019

Don't worry we update plugin before stable release webpack@5, right now the plugin doesn't work?

@andriijas
Copy link
Author

@evilebottnawi Thanks! Im not worried :) There are a couple of deprecation warnings and I would appreciate help and/or feedback to get a beta of this plugin without the warnings for integration in facebook/create-react-app#7929

Thanks 🙌

@alexander-akait
Copy link
Member

@andriijas just ignore deprecation messages right now, it is not break code or something important

@alexander-akait
Copy link
Member

Compatibility is fixed, release will be soon

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