atom-npm-install icon indicating copy to clipboard operation
atom-npm-install copied to clipboard

consider using npm package directly instead of spawning a new process

Open capaj opened this issue 11 years ago • 3 comments

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?

capaj avatar Mar 16 '15 17:03 capaj

hmmmmm

abdulhannanali avatar Apr 19 '15 08:04 abdulhannanali

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 avatar May 16 '15 01:05 hughsk

@hughsk error handling should be easier, because you don't have to parse the command line errors. How do you handle errors now?

capaj avatar May 16 '15 09:05 capaj