Skip to content
This repository has been archived by the owner on Feb 18, 2024. It is now read-only.

"jspm install npm:three" bug #41

Closed
speigg opened this issue Jan 27, 2015 · 4 comments
Closed

"jspm install npm:three" bug #41

speigg opened this issue Jan 27, 2015 · 4 comments

Comments

@speigg
Copy link

speigg commented Jan 27, 2015

Unnecessarily adds Buffer and process as dependencies.

@guybedford
Copy link
Member

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
Copy link
Member

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

@speigg
Copy link
Author

speigg commented Feb 1, 2015

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 guybedford added bug and removed bug labels Apr 19, 2015
@guybedford
Copy link
Member

Duplicate of #55.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants