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

Bump webpack-dev-server #7988

Merged
merged 4 commits into from
Nov 24, 2019
Merged

Bump webpack-dev-server #7988

merged 4 commits into from
Nov 24, 2019

Conversation

ianschmitz
Copy link
Contributor

@ianschmitz ianschmitz commented Nov 16, 2019

Closes #7782.

Updates webpack-dev-server from 3.2.1 to 3.9.0.

I was able to resolve the previous issues i experienced when trying the upgrade before in #7473. The error appears to be related to our usage of sockjs-client inside of the webpackHotDevClient and jsdom. Switching to using native a WebSocket instead made everything happy.

yarn start works nicely. Code changes cause page refresh, style changes causes a hot replacement of styles as expected, and error overlay works.

I tagged as a breaking change for react-dev-utils only as this change could potentially affect someone out there due to the change.

Copy link
Contributor

@andriijas andriijas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@heygrady
Copy link
Contributor

This fixes the failing tests on #7750

@ianschmitz ianschmitz merged commit 82009f5 into facebook:master Nov 24, 2019
@ianschmitz ianschmitz deleted the bump-wds branch November 24, 2019 22:18
@lock lock bot locked and limited conversation to collaborators Nov 29, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Incompatible SockJS! Main site uses: "1.4.0", the iframe: "1.3.0".
4 participants