[eslint] Don't treat eslint warnings as PRESUBMIT errors

catapult_build/js_checks.py

Index: catapult_build/js_checks.py
diff --git a/catapult_build/js_checks.py b/catapult_build/js_checks.py
index 5c6b68716070e81ad1ba0ba3f7385cc76b1b559d..2af873dbf75a26c1909e5a79dbed1c74ee9cb11a 100644
--- a/catapult_build/js_checks.py
+++ b/catapult_build/js_checks.py
@@ -84,10 +84,10 @@ class JSChecker(object):
         error_lines += filter(None, [self.ConstCheck(i, line)])
 
     if affected_js_files:
-      eslint_output = eslint.RunEslintOnFiles(
-          [f.AbsoluteLocalPath() for f in affected_js_files]).rstrip()
+      success, eslint_output = eslint.RunEslint(
+          [f.AbsoluteLocalPath() for f in affected_js_files])
 
-      if eslint_output:
+      if not success:
         error_lines.append('\neslint found lint errors:')
         error_lines.append(eslint_output)