Skip to content

module: remove dead code

Rodrigo Muino Tomonari requested to merge github/fork/BridgeAR/remove-module-dead-code into master

This removes a lot of code that has no functionality anymore. All Node.js internal code calls _resolveLookupPaths with two arguments.

The code that validates index.js is not required at all as we check for these files anyway, so it's just redundant code that should be removed.

I defensively marked this as semver-major even though I did not find any usage of the third argument outside of Node.js and that would only be an issue if that part was monkey patched.

Refs: #25362

@nodejs/modules PTAL

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

Merge request reports