-
-
Notifications
You must be signed in to change notification settings - Fork 9.1k
HMR support for ESM #19616
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
HMR support for ESM #19616
Conversation
CodSpeed Performance ReportMerging #19616 will degrade performances by 69.28%Comparing Summary
Benchmarks breakdown
|
8b4c52b
to
422d7f4
Compare
422d7f4
to
5e68bc4
Compare
5e68bc4
to
f64aabb
Compare
Hi @xiaoxiaojx, thank you for taking over. Is there a way to contact you directly? I'd like to discuss something with you. Thanks and the implementation looks solid overall! |
Hello @3ru , You can reach me on Telegram (natsu_xiao) or WeChat (natsu_xiao) . I’d be happy to chat about webpack anytime. |
What kind of change does this PR introduce?
feature: HMR support for ESM. Original PR #19561
Did you add tests for your changes?
Yes
Does this PR introduce a breaking change?
No
What needs to be documented once your changes are merged?
Yes