-
Notifications
You must be signed in to change notification settings - Fork 29.8k
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
esm: fix loading of (recursive) CJS modules from ESM #49500
Conversation
Review requested:
|
Fast-track has been requested by @aduh95. Please 👍 to approve. |
Approving fast-track per: node/doc/contributing/collaborator-guide.md Line 186 in d8eb8d0
introduced in 20.6.0 released yesterday |
Landed in fdc65d9 |
This is the release schedule (expectation dates) |
Thank you! It looks like there are only planned dates for the next minor versions there. @UlisesGascon I see that you released 20.6.0 and are working on 20.7.0, maybe you can help here? :) |
I'll create the v20.6.1 proposal. But, I won't be able to sign and promote (I'm travelling) - ping @nodejs/releasers. |
Notable changes: esm: * fix loading of CJS modules from ESM (Antoine du Hamel) #49500 PR-URL: TBD
I have bandwidth for it @RafaelGSS @nicolo-ribaudo, I can promote it but I will need first to unblock nodejs/Release#892 🙂 |
PR-URL: nodejs#49500 Fixes: nodejs#49497 Reviewed-By: Stephen Belanger <admin@stephenbelanger.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: Jacob Smith <jacob@frende.me>
Notable changes: esm: * fix loading of CJS modules from ESM (Antoine du Hamel) nodejs#49500 PR-URL: nodejs#49528
Fixes: #49497