antonmedv / jsize

Find out minified and gzipped npm package size
MIT License
178 stars 12 forks source link

Auto-switch targets #37

Closed antonmedv closed 7 years ago

antonmedv commented 7 years ago

When we get error like this:

Error: Can't resolve 'module'

Automatically calculate again with -t node.

This will help a lot of users.

DylanPiercey commented 7 years ago

What is an example package that has this error? I haven't seen this happen before.

antonmedv commented 7 years ago

Try this – jsize r2

DylanPiercey commented 7 years ago

r2 includes gulp which is a bit complicated. In fact it doesn't compile properly regardless of target, not 100% sure why yet.

Also IMO it should not auto switch targets as this is confusing for the user and there is no possible way to know for sure the intended target, I'd say more helpful error messages are best.