commit e245d1a20821d9e32cfd7f14d63e525aa1eed2e7
author Jack Franklin <jacktfranklin@chromium.org> Thu Feb 13 15:25:13 2020 +0000
committer Commit Bot <commit-bot@chromium.org> Fri Feb 14 11:58:54 2020 +0000
tree 0c4d3518fb75911efacf56520280bb101d1ff5ca
parent 15d4c9308d6bef2679ad80f2e5e236790877c5fc

Include `test/e2e` in ESLint checks.

It already runs on `test/unittests` but
with this change we'll lint the e2e tests too.

Change-Id: I0e39ce9a4b0128207a04b1dfd6cf1ea2f25bf213
Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/2054159
Reviewed-by: Paul Lewis <aerotwist@chromium.org>
Reviewed-by: Tim van der Lippe <tvanderlippe@chromium.org>
Commit-Queue: Jack Franklin <jacktfranklin@chromium.org>

PRESUBMIT.py

diff --git a/PRESUBMIT.py b/PRESUBMIT.py
index 63aba0c..a359beb 100644
--- a/PRESUBMIT.py
+++ b/PRESUBMIT.py
@@ -118,7 +118,8 @@
     eslint_ignore_path = input_api.os_path.join(input_api.PresubmitLocalPath(), '.eslintignore')
     with open(eslint_ignore_path, 'r') as ignore_manifest:
         for line in ignore_manifest:
-            ignore_files.append(input_api.os_path.normpath(line.strip()))
+            if '*' not in line.strip():
+                ignore_files.append(input_api.os_path.normpath(line.strip()))
     formattable_files = [
         affected_file for affected_file in affected_files if all(ignore_file not in affected_file for ignore_file in ignore_files)
     ]