Skip to content

doc: update experimental loader hooks example code

Rodrigo Muino Tomonari requested to merge github/fork/zaverden/patch-1 into master

It fix 2 issues in provided Loader hooks examples:

  1. Original new URL(`${process.cwd()}/`, 'file://'); is not cross-platform, it gives wrong URL on windows
  2. Based on CHECK in ModuleWrap::Resolve (node 12.9.1, https://github.com/nodejs/node/blob/v12.9.1/src/module_wrap.cc#L1132) the 2nd parameter should be a string, not an URL object
Checklist

Merge request reports

Loading