-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
fix-2239 : Add webpack@5 in CI #2320
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
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2320 +/- ##
=======================================
Coverage 93.92% 93.92%
=======================================
Files 34 34
Lines 1284 1284
Branches 366 366
=======================================
Hits 1206 1206
Misses 77 77
Partials 1 1 Continue to review full report at Codecov.
|
Sorry, I am not sure which version to follow/checkout, The tests are failing for webpack@5 |
@@ -50,6 +50,9 @@ jobs: | |||
node-6: | |||
node_version: ^6.9.0 | |||
webpack_version: latest | |||
webpack-new: | |||
node_version: ^10.13.0 | |||
webpack_version: 5.0.0-alpha.26 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
next
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry , but I just found out this pr is same as #2244, since the issue was not tagged from other pr , I end up making a pr .
I will go ahead and close it :).
Thanks for your time
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
also CLI is broken for webpack@5, we should wait webpack-cli
release or create workaround for bin
For Bugs and Features; did you add new tests?
NA
Motivation / Use-Case
Handling : #2239, @evilebottnawi : I tried setting up CI for webpack@5 according to #2239 (comment)
Breaking Changes
NA
Additional Info
NA