-
-
Notifications
You must be signed in to change notification settings - Fork 31.9k
[v22.x backport] esm: implement import.meta.main #58693
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
base: v22.x-staging
Are you sure you want to change the base?
[v22.x backport] esm: implement import.meta.main #58693
Conversation
Review requested:
|
Should include #58661 if it doesn't already |
We should only pick the commits that have landed on |
Ah, my bad, I probably picked commits from the wrong branch. |
Boolean value to check if an ES Module is the entrypoint of the current process. Implements: nodejs#57226 Co-authored-by: Antoine du Hamel <duhamelantoine1995@gmail.com> PR-URL: nodejs#57804 Fixes: nodejs#57226 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com> Reviewed-By: Guy Bedford <guybedford@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Marco Ippolito <marcoippolito54@gmail.com> (cherry picked from commit bbc0593)
PR-URL: nodejs#58661 Fixes: nodejs#58660 Reviewed-By: Xuguang Mei <meixuguang@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: Jacob Smith <jacob@frende.me> Reviewed-By: Ethan Arrowood <ethan@arrowood.dev> Reviewed-By: Pietro Marchini <pietro.marchini94@gmail.com> Reviewed-By: Chemi Atlow <chemi@atlow.co.il> Reviewed-By: Yagiz Nizipli <yagiz@nizipli.com> (cherry picked from commit bba07d7)
`phase` parameter doesn't exist in v22.x
e529a26
to
8868f38
Compare
@aduh95 to adhere to the 'only commits that landed' rule, should I squash the new commits into a landed commit? or are commits for manual resolution after cherry-picking OK? |
What's the deal here? I don't understand why CI is failing 🤔 |
Backport #57804 to v22.x release line