[v22.x backport] esm: implement import.meta.main
Backport #57804 to v22.x release line
Review requested:
- [ ] @nodejs/loaders
Should include https://github.com/nodejs/node/pull/58661 if it doesn't already
We should only pick the commits that have landed on main, this PR have 47 commits that are not on main
Ah, my bad, I probably picked commits from the wrong branch.
@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 🤔
@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?
You can use git commit --fixup <commit-sha>, so it's easier to review and squash those additional commits upon landing.
What's the deal here? I don't understand why CI is failing 🤔
It looks like the Coverage CI is broken on that branch, unrelated to your changes. The macOS one timed out, also probably unrelated to your changes.
can you include https://github.com/nodejs/node/commit/bba07d7e1edace182894b517a903cb55fb4d5b2b in the backport (if lands cleanly otherwise I can open a backport)
can you include https://github.com/nodejs/node/commit/bba07d7e1edace182894b517a903cb55fb4d5b2b in the backport (if lands cleanly otherwise I can open a backport)
'In addition to' or 'instead of' d8d3b07 (which is currently included) ?
Sorry I didnt notice it was already included. Looks good
What's the deal here? I don't understand why CI is failing 🤔
It looks like the Coverage CI is broken on that branch, unrelated to your changes.
FWIW the broken Coverage Windows GitHub Workflow is the same breakage as https://github.com/nodejs/node/issues/58801 (and unrelated to this PR). cc FYI @nodejs/releasers
CI: https://ci.nodejs.org/job/node-test-pull-request/68064/
CI: https://ci.nodejs.org/job/node-test-pull-request/68088/
Rebased onto v22.x-staging
Can you please rebase?
Something weird happened and I had to drop a couple of commits, then redo the rebase - let me know whether or not things look OK
Rebased on top of v22.x, fixed the commit message
CI: https://ci.nodejs.org/job/node-test-pull-request/68251/
CI: https://ci.nodejs.org/job/node-test-pull-request/68257/
Landed in f99aa748c090...2fc89892abaa