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

Upgrade to Webpack 4 + Babel 7 #1003

Merged
merged 14 commits into from
Apr 30, 2019
Merged

Conversation

davidgolden
Copy link
Contributor

Upgrades webpack to 4.x and babel to 7.x. This addresses #1000, and fixes #970, #807, and #237.

…anifest. Replace .babelrc with babel.config.js in react-server-test-pages. Utilize the react-hot-loader babel plugin instead of the webpack loader. Update the generator-react-server template files and associated tests.
… test again. Ensure we're actually killing the test process with rs.kill()!!
…anifest. Replace .babelrc with babel.config.js in react-server-test-pages. Utilize the react-hot-loader babel plugin instead of the webpack loader. Update the generator-react-server template files and associated tests.
… test again. Ensure we're actually killing the test process with rs.kill()!!
…ade-webpack-babel

# Conflicts:
#	packages/react-server-core-middleware/package.json
@davidgolden davidgolden changed the title Upgrade webpack babel Upgrade to Webpack 4 + Babel 7 Apr 24, 2019
@drewpc drewpc requested review from doug-wade and gigabo April 24, 2019 15:26
@drewpc drewpc added breaking change enhancement New functionality. labels Apr 24, 2019
@drewpc drewpc added this to the 1.0 milestone Apr 24, 2019
@drewpc
Copy link
Collaborator

drewpc commented Apr 24, 2019

This builds on #1001 and #1002, so once those PRs are merged, this will seem a lot less daunting. Please approve #1001 and #1002 first.

Note that this includes some breaking changes that we can address in the release documentation/change log.

Also removes the need for #932

@drewpc
Copy link
Collaborator

drewpc commented Apr 30, 2019

Merging per conversation with @roblg.

@drewpc drewpc merged commit 137c20e into redfin:master Apr 30, 2019
@drewpc drewpc removed the enhancement New functionality. label May 2, 2019
@drewpc drewpc deleted the upgrade-webpack-babel branch May 10, 2019 20:29
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.

Client-side hot module replacement does not work
3 participants