search

browserify / commonjs-assert

Node.js's require('assert') for all engines
MIT License
293 stars 57 forks source link

Update all `require('util/')` to be `require('util') because the trailing slash isn't needed #59

Closed sstchur closed 1 year ago

sstchur commented 1 year ago

While working on a project using a new bundler, we discovered that the presence of the trailing slash is breaking the bundler. Since that trailing slash isn't needed, I'm proposing removing it from all require('util') statements.

ljharb commented 1 year ago

The trailing slash is absolutely needed - without the slash, it's the core module, with it, it's the util package from npm.

If that's breaking your bundler, then your bundler is broken.