search

jspm / npm

NPM Location Service
19 stars 34 forks source link

"jspm install npm:three" bug #41

Closed speigg closed 9 years ago

speigg commented 9 years ago

Unnecessarily adds Buffer and process as dependencies.

guybedford commented 9 years ago

Thanks - ok we can improve this by detecting the use of Buffer or Process using proper scope analysis on the tree instead of regular expressions.

guybedford commented 9 years ago

Let me know if this seems urgent, but for now I'm prioritising other issues first.

speigg commented 9 years ago

That's fine, I can use github:mrdoob/three for now, but npm would be nice eventually as it just downloads the build files.

guybedford commented 9 years ago

Duplicate of https://github.com/jspm/npm/issues/55.