Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3)

Unified Diff: tools/presubmit.py

Issue 2611793002: [test] Process to locally develop and upstream test262 tests (Closed)
Patch Set: Cleanup nit Created 3 years, 11 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « test/test262/upstream-local-tests.sh ('k') | no next file » | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: tools/presubmit.py
diff --git a/tools/presubmit.py b/tools/presubmit.py
index fce940a2d8f8b6fcdf8721600fa28f33c3b72c33..a17839ec081e9ba3b54eb2e54af142a6046c593d 100755
--- a/tools/presubmit.py
+++ b/tools/presubmit.py
@@ -359,6 +359,8 @@ class SourceProcessor(SourceFileProcessor):
'zlib.js']
IGNORE_TABS = IGNORE_COPYRIGHTS + ['unicode-test.js', 'html-comments.js']
+ IGNORE_COPYRIGHTS_DIRECTORY = "test/test262/local-tests"
+
def EndOfDeclaration(self, line):
return line == "}" or line == "};"
@@ -374,7 +376,8 @@ class SourceProcessor(SourceFileProcessor):
if '\t' in contents:
print "%s contains tabs" % name
result = False
- if not base in SourceProcessor.IGNORE_COPYRIGHTS:
+ if not base in SourceProcessor.IGNORE_COPYRIGHTS and \
+ not SourceProcessor.IGNORE_COPYRIGHTS_DIRECTORY in name:
if not COPYRIGHT_HEADER_PATTERN.search(contents):
print "%s is missing a correct copyright header." % name
result = False
« no previous file with comments | « test/test262/upstream-local-tests.sh ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698