search

browserify / browser-resolve

resolve function which support the browser field in package.json
MIT License
102 stars 70 forks source link

No support for npm module name with "scope" yet? #77

Closed liujingbreak closed 8 years ago

liujingbreak commented 8 years ago

Looks like it does not support it yet? module name liks @someScope/my-module https://docs.npmjs.com/misc/scope

defunctzombie commented 8 years ago

I have not tried it but would welcome a PR that adds support.

liujingbreak commented 8 years ago

Oh, my bad just again, tested again. New results

The target module's package.json file only has "browser" field, no "main" field. Then, it works in sync version resolve.sync('@xx/yy') but failed to find that module in async way resolve('@xx/yy', callback)

liujingbreak commented 8 years ago

Issue gone, I can not reproduce it again, probably it's my bad.