doc, esm: correct default calls in loader examples (!44708) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/yorkie/doc/fix-esm-examples into main Sep 18, 2022

The default ESM functions load and resolve does require the context parameter, see:

defaultLoad https://github.com/nodejs/node/blob/v18.9.0/lib/internal/modules/esm/load.js#L72-L100
defaultResolve https://github.com/nodejs/node/blob/v18.9.0/lib/internal/modules/esm/resolve.js#L984

I found this because of nested dependency loading when I'm writing a custom loader, so I think we need to fix it in the document so other users don't have the same problem :)