Lint command fails if there are any warnings (#10572)

You get a lint failure! You get a lint failure! Eeeeverybody gets a lint failure!
facebook · Aug 30, 2017 · 9917132 · 9917132
1 parent cd52f7e
commit 9917132
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/scripts/tasks/eslint.js b/scripts/tasks/eslint.js
@@ -14,14 +14,19 @@ var spawn = require('child_process').spawn;
 
 var extension = process.platform === 'win32' ? '.cmd' : '';
 
-spawn(path.join('node_modules', '.bin', 'eslint' + extension), ['.'], {
-  // Allow colors to pass through
-  stdio: 'inherit',
-}).on('close', function(code) {
+spawn(
+  path.join('node_modules', '.bin', 'eslint' + extension),
+  ['.', '--max-warnings=0'],
+  {
+    // Allow colors to pass through
+    stdio: 'inherit',
+  }
+).on('close', function(code) {
   if (code !== 0) {
     console.error('Lint failed');
+  } else {
+    console.log('Lint passed');
   }
 
-  console.log('Lint passed');
   process.exit(code);
 });