Support browser path false for require without ext

rollup / rollup-plugin-node-resolve

This module has moved and is now available at @rollup/plugin-node-resolve / https://github.com/rollup/plugins/blob/master/packages/node-resolve

MIT License

469 stars 96 forks source link

Support browser path false for require without ext #195

Closed porsager closed 4 years ago

porsager commented 5 years ago

It's pretty common for node modules to use require() without specifying the file extension, so adding this will most likely make a few more libraries work with rollup. One project doing this is for instance object-assign which doesn't work currently. https://github.com/substack/object-inspect/issues/18

Considering the line that checks for changed paths below does exactly the same it's probably just an oversight that this was missing.

shellscape commented 4 years ago

Closing as abandoned. We're getting ready to move this plugin to https://github.com/rollup/plugins and would welcome a new PR there for review once the move is complete. :beer: