atom-npm-install
atom-npm-install copied to clipboard
consider using npm package directly instead of spawning a new process
wouldn't it make a little more sense to use npm.install: https://github.com/npm/npm/blob/master/doc/api/npm-install.md instead of spawning a new process?
hmmmmm
npm, as wonderful as it is, has all kinds of things going on in its codebase. Using a separate process insulates your editor from that, and I think it's best to stick that way, unless there's a good reason to switch to using the package directly?
@hughsk error handling should be easier, because you don't have to parse the command line errors. How do you handle errors now?