vadimdemedes/gifi

Handle npm errors.

Open

#28 ouverte le 27 avr. 2016

Voir sur GitHub
 (3 commentaires) (0 réactions) (0 assignés)JavaScript (1 025 stars) (42 forks)batch import
enhancementhelp wanted

Description

Had a syntax error in a package.json. Took a few tries to realize I had the error with packages not downloading. Would be nice if Gifi printed out that error when it happens instead of just exiting with no message.

Guide contributeur

Stack technique
nodejs
Domaine
developer experiencetooling
Type d'issue
feature
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
2
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
mostly clear
Prérequis
JavaScriptNode.js child process
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
70
Direction de recherche
The issue requests that gifi display npm error messages when npm install fails due to package.json syntax errors. Investigate how gifi currently spawns npm (likely using child process.spawn or exec) and capture stderr output. Modify the relevant file (probably index.js or a similar entry point) to print any error output from npm to the console before exiting. Consider using the 'exit' event or checking the stderr buffer. The existing comments may provide hints.