Skip to content

Allow v5 of webpack-dev-server as peer dependency#836

Closed
mxdvl wants to merge 6 commits intopmmmwh:mainfrom
mxdvl:patch-1
Closed

Allow v5 of webpack-dev-server as peer dependency#836
mxdvl wants to merge 6 commits intopmmmwh:mainfrom
mxdvl:patch-1

Conversation

@mxdvl
Copy link
Copy Markdown

@mxdvl mxdvl commented Apr 22, 2024

Release notes v5.0.0 along with the migration guide.

  • The Node version of the consumer is not a concern of this package
  • The webpack and webpack-cli of the consumer are not a concern of this package
  • The http & https options are not used internally

Should we also bump the local dev dependency on webpack-dev-server to v5?

@codesandbox
Copy link
Copy Markdown

codesandbox bot commented Apr 22, 2024

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@pmmmwh
Copy link
Copy Markdown
Owner

pmmmwh commented Apr 25, 2024

Should we also bump the local dev dependency on webpack-dev-server to v5?

I think we should, just to make sure tests pass. If that can be done I'm happy to cut this release first then work towards moving completely over to v5 👍🏻

@pmmmwh
Copy link
Copy Markdown
Owner

pmmmwh commented Apr 25, 2024

Fixes #787
Fixes #800
Fixes #824

mxdvl added 3 commits April 25, 2024 09:27
this ensures the test pass locally,
but does not address how to keep
tests running against v4
in CI, we have incompatible packes,
but that is acceptable as we are testing
different Node versions againt the library
@mxdvl
Copy link
Copy Markdown
Author

mxdvl commented Apr 25, 2024

@pmmmwh I’ve tried and failed to make CI pass:

  • the minimum Node version is 18.12, and it seems your current puppeteer image is 18.9, so that fails for webpack 4 and webpack 5
  • the test server is also failing to run… I am experiencing the same thing locally and feel like I am not familiar enough with the project to make this work

@pmmmwh pmmmwh mentioned this pull request Apr 27, 2024
@pmmmwh pmmmwh closed this in #837 Apr 27, 2024
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.

2 participants