Skip to content

Commit

Permalink
doc: esm loader example with module.builtinModules
Browse files Browse the repository at this point in the history
PR-URL: #17385
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Jon Moss <[email protected]>
Reviewed-By: Timothy Gu <[email protected]>
  • Loading branch information
guybedford authored and MylesBorins committed Dec 12, 2017
1 parent 8c21430 commit b015747
Showing 1 changed file with 3 additions and 5 deletions.
8 changes: 3 additions & 5 deletions doc/api/esm.md
Original file line number Diff line number Diff line change
Expand Up @@ -147,15 +147,13 @@ be written:
import url from 'url';
import path from 'path';
import process from 'process';
import Module from 'module';

const builtins = new Set(
Object.keys(process.binding('natives')).filter((str) =>
/^(?!(?:internal|node|v8)\/)/.test(str))
);
const builtins = Module.builtinModules;
const JS_EXTENSIONS = new Set(['.js', '.mjs']);

export function resolve(specifier, parentModuleURL/*, defaultResolve */) {
if (builtins.has(specifier)) {
if (builtins.includes(specifier)) {
return {
url: specifier,
format: 'builtin'
Expand Down

0 comments on commit b015747

Please sign in to comment.