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
Update polyfills to sync with latest RN #414
Conversation
Hold on for now, I didn't apply some of my changes |
// | ||
if (!global.self) { | ||
global.self = global; /* eslint-disable-line */ | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We now inject self
as a banner, as with Webpack 4 upgrade we seem to need to act on it earlier
This is ready for review now |
} | ||
ErrorUtils.applyWithGuard(fun, context, args); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That's the magic? 💛
@thymikee I will run the tests ASAP. I will get back to you then! Thank you for finish this off. 🙏 |
"webpack-dev-middleware": "^1.12.0", | ||
"webpack-hot-middleware": "^2.19.1", | ||
"webpack": "^4.6.0", | ||
"webpack-hot-middleware": "^2.22.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wut?? We don't use it anymore.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
will remove then :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually we do, for process-update:
haul/src/hot/client/hotClient.js
Line 9 in 95ade60
const processUpdate = require('webpack-hot-middleware/process-update'); |
we use process-update from webpack-hot-middleware
Builds on top of #410 (thanks @jukben), but without the breaking changes.
We decided to not introduce breakage (apart from new config, which is breaking gracefully) for now. We can however update the polyfills, which simplifies a bit our BannerPlugin usage.
I've tested
example-app
linked with yarn on both iOS and Android with live reload and HMR.