Bug 1307622: Fix handling of ignored file warnings. r?Mossop draft
authorKris Maglione <maglione.k@gmail.com>
Tue, 04 Oct 2016 22:44:50 +0100
changeset 420868 88a75a254b7a9b57c89a12b1778219c64f2f6898
parent 420867 672ff5d3b753a8e1240e009ce9eb972fdf05a0d1
child 532920 82af1aa8f8b47f469d1e0b7c182cce962883c680
push id31322
push usermaglione.k@gmail.com
push dateTue, 04 Oct 2016 21:47:17 +0000
reviewersMossop
bugs1307622
milestone52.0a1
Bug 1307622: Fix handling of ignored file warnings. r?Mossop MozReview-Commit-ID: EV3YPfsSSDS
tools/mercurial/eslintvalidate.py
--- a/tools/mercurial/eslintvalidate.py
+++ b/tools/mercurial/eslintvalidate.py
@@ -3,30 +3,33 @@
 
 import os
 import sys
 import re
 import json
 from subprocess import check_output, CalledProcessError
 
 lintable = re.compile(r'.+\.(?:js|jsm|jsx|xml|html)$')
-ignored = "File ignored because of your .eslintignore file. Use --no-ignore to override."
+ignored = 'File ignored because of a matching ignore pattern. Use "--no-ignore" to override.'
 
 def is_lintable(filename):
     return lintable.match(filename)
 
 def display(ui, output):
     results = json.loads(output)
     for file in results:
         path = os.path.relpath(file["filePath"])
         for message in file["messages"]:
             if message["message"] == ignored:
                 continue
 
-            ui.warn("%s:%d:%d %s\n" % (path, message["line"], message["column"], message["message"]))
+            if "line" in message:
+                ui.warn("%s:%d:%d %s\n" % (path, message["line"], message["column"], message["message"]))
+            else:
+                ui.warn("%s: %s\n" % (path, message["message"]))
 
 def eslinthook(ui, repo, node=None, **opts):
     ctx = repo[node]
     if len(ctx.parents()) > 1:
         return 0
 
     deleted = repo.status(ctx.p1().node(), ctx.node()).deleted
     files = [f for f in ctx.files() if f not in deleted and is_lintable(f)]