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

Release v1.13 with webpack v4 listed as peedDependency #263

Closed
1 of 2 tasks
mikegreiling opened this issue Feb 25, 2018 · 5 comments
Closed
1 of 2 tasks

Release v1.13 with webpack v4 listed as peedDependency #263

mikegreiling opened this issue Feb 25, 2018 · 5 comments

Comments

@mikegreiling
Copy link
Member

  • Operating System: macOS 10.13.3
  • Node Version: v9.5.0
  • NPM Version: v5.6.0
  • webpack version: 4.0.0
  • webpack-dev-middleware Version: 1.12.1
  • This is a feature request
  • This is a bug

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

  "peerDependencies": {
    "webpack": "^1.0.0 || ^2.0.0 || ^3.0.0"
  },

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 the peerDependencies.

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:

error "karma-webpack#webpack-dev-middleware#webpack@^1.0.0 || ^2.0.0 || ^3.0.0" doesn't satisfy found match of "[email protected]"

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. Since karma-webpack is still using v1.12.x, it is holding us back from upgrading to webpack v4 🙁

@mikegreiling
Copy link
Member Author

Related discussion in the karma-webpack library:
codymikol/karma-webpack#301 (comment)

@mikegreiling
Copy link
Member Author

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:

  1. create a branch off of tag v1.12.2 called something like v1-stable
  2. bump the version to v1.13.0
  3. add || ^4.0.0 to the webpack peerDependency
  4. tag and publish the new version 😄

@shellscape
Copy link
Contributor

shellscape commented Feb 25, 2018

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.

@shellscape
Copy link
Contributor

found a spare moment and created the v1-stable branch. please do submit a PR against that branch. the old v1 tests will have to pass against webpack v4 so please be sure to run those tests with that installed.

@mikegreiling
Copy link
Member Author

I've created #265 to address this. Thanks @shellscape!

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

No branches or pull requests

2 participants