fix(refactor): replace console.error with process.stderr

watilde · Aug 3, 2017 · ad5df92 · ad5df92
1 parent e12d136
commit ad5df92
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/bin/dep.js b/bin/dep.js
@@ -9,8 +9,8 @@ const pkgJSON = require('../package.json')
 const notifier = updateNotifier({pkg: pkgJSON})
 
 if (!semver.satisfies(process.version, pkgJSON.engine.node)) {
-  console.error('dep works only on Node.js LTS versions')
-  console.error('See the schedule: https://github.com/nodejs/LTS#lts-schedule1')
+  process.stderr.write('dep works only on Node.js LTS versions\n')
+  process.stderr.write('See the schedule: https://github.com/nodejs/LTS#lts-schedule1\n')
   process.exit(1)
 }
 

diff --git a/lib/install.js b/lib/install.js
@@ -42,8 +42,8 @@ const install = (argv) => {
       process.stdout.write(
         `Installed ${global.dependenciesCount} packages in ${s}s\n`
       )
-    }).catch(console.error)
-  }).catch(console.error)
+    }).catch((e) => { process.stderr(e.stack) })
+  }).catch((e) => { process.stderr(e.stack) })
 }
 
 module.exports = {

diff --git a/lib/lock.js b/lib/lock.js
@@ -21,7 +21,7 @@ const lock = (argv) => {
     process.stdout.write(
       `created node_modules.json\n`
     )
-  }).catch(console.error)
+  }).catch((e) => { process.stderr(e.stack) })
 }
 
 module.exports = {