-
-
Notifications
You must be signed in to change notification settings - Fork 377
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
Release v1.13 with webpack v4 listed as peedDependency #263
Comments
Related discussion in the karma-webpack library: |
I'd open a PR for this, but it doesn't look like there is a v1.x branch I can target. The process would be simple though:
|
This sounds entirely reasonable to me. Thanks for creating the issue for it. It may take me more than today to get to, but I'll get a branch up for 1.x maintenance as soon as I'm able to. |
found a spare moment and created the |
I've created #265 to address this. Thanks @shellscape! |
Code
The latest 1.x release (v1.12.2) will raise warnings when installed alongside webpack v4
https://github.com/webpack/webpack-dev-middleware/blob/v1.12.2/package.json#L7
Expected Behavior
Since webpack-dev-middleware v1.12.2 appears to be compatible with webpack v4.0.0, and since several webpack plugins still have not been updated to use webpack-dev-middleware v2.x, I think it would be beneficial to release v1.13.0 which adds
^4.0.0
to thepeerDependencies
.Actual Behavior
When a project updates to webpack v4 and another dependency relies on webpack-dev-middleware ^1.12.0 as a sub-dependency, warnings will be thrown by most package managers and
yarn check
will fail with an error like so:For Bugs; How can we reproduce the behavior?
N/A
For Features; What is the motivation and/or use-case for the feature?
I realize the best long-term goal is to nudge webpack libraries to update their webpack-dev-middleware dependencies, but for now, releasing a minor version bump for those stuck with v1.x would be a big benefit.
In my company's workflow CI will not pass if
yarn check
throws errors, so our peerDependencies have to match. Sincekarma-webpack
is still using v1.12.x, it is holding us back from upgrading to webpack v4 🙁The text was updated successfully, but these errors were encountered: