Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Adapt to Webpack 5#114

Merged
developit merged 1 commit intodevelopit:master from
kenrick95:kenrick/webpack-5
May 23, 2022
Merged

Adapt to Webpack 5 #114
developit merged 1 commit intodevelopit:master from
kenrick95:kenrick/webpack-5

Conversation

@kenrick95
Copy link
Contributor

@kenrick95 kenrick95 commented Jan 13, 2021
edited
Loading

Attempt to support Webpack 5

Fixes #77

I'm aware of #101 but when I use it on my application, it produces some runtime error that I couldn't understand


Note that this still emits a deprecating warning ("Compilation.cache was removed in favor of Compilation.getCache()") due to usage of compilation.cache, but I'm not too sure how to use Compilation.getCache since it is lacking documentation (webpack/webpack.js.org#3843)


P.S. The tests aren't migrated to webpack 5 yet since karmatic haven't had support for it yet (karma-webpack has it but still in alpha stage)

naoak, niklaskorz, danielterwiel, paulo-goncalves, jljorgenson18, dcosta-ptc, johh, EvgeneOskin, and cvn reacted with thumbs up emoji
@kenrick95 kenrick95 force-pushed the kenrick/webpack-5 branch 3 times, most recently from 7fde4cc to d439b8a Compare January 13, 2021 04:00
Copy link

Is this project dead? This Pull Request would be a great addition if this project is intended to be alive :)

paulo-goncalves, johh, and cullylarson reacted with thumbs up emoji

Copy link

It must be because webpack 5 supports this feature.
https://webpack.js.org/guides/web-workers/

developit reacted with thumbs up emoji

@developit developit merged commit 3f93c86 into developit:master May 23, 2022
Copy link
Owner

My apologies, I use Webpack so rarely these days and the API has changed so much that it's been difficult to keep up with merging things. This fix was released as 2.0.1.

EvgeneOskin and naoak reacted with thumbs up emoji

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Webpack 5 support

AltStyle γ«γ‚ˆγ£γ¦ε€‰ζ›γ•γ‚ŒγŸγƒšγƒΌγ‚Έ (->γ‚ͺγƒͺγ‚ΈγƒŠγƒ«) /