check-style should not complain about code in icu directories
authordbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Dec 2016 03:35:54 +0000 (03:35 +0000)
committerdbates@webkit.org <dbates@webkit.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Fri, 16 Dec 2016 03:35:54 +0000 (03:35 +0000)
https://bugs.webkit.org/show_bug.cgi?id=53929

Reviewed by Darin Adler.

Skip style checking of imported ICU headers to avoid a deluge of style issues
as they do not conform to the WebKit Code Style guideline. We want to preserve
the formatting of these headers so as to make it straightforward to merge
changes to them, if necessary.

* Scripts/webkitpy/style/checker.py:

git-svn-id: http://svn.webkit.org/repository/webkit/trunk@209902 268f45cc-cd09-0410-ab3c-d52691b4dbfc

Tools/ChangeLog
Tools/Scripts/webkitpy/style/checker.py

index 52d2d3e..81146f6 100644 (file)
@@ -1,3 +1,17 @@
+2016-12-15  Daniel Bates  <dabates@apple.com>
+
+        check-style should not complain about code in icu directories
+        https://bugs.webkit.org/show_bug.cgi?id=53929
+
+        Reviewed by Darin Adler.
+
+        Skip style checking of imported ICU headers to avoid a deluge of style issues
+        as they do not conform to the WebKit Code Style guideline. We want to preserve
+        the formatting of these headers so as to make it straightforward to merge
+        changes to them, if necessary.
+
+        * Scripts/webkitpy/style/checker.py:
+
 2016-12-15  Alexey Proskuryakov  <ap@apple.com>
 
         Move JSC testers from Yosemite to Sierra
index e2fa4fd..4384a18 100644 (file)
@@ -315,8 +315,16 @@ _SKIPPED_FILES_WITH_WARNING = [
 # with FileType.NONE are automatically skipped without warning.
 _SKIPPED_FILES_WITHOUT_WARNING = [
     "LayoutTests" + os.path.sep,
+
     # Files generated by the bindings script should not be checked for style.
-    os.path.join('Source', 'WebCore', 'bindings', 'scripts', 'test')]
+    os.path.join('Source', 'WebCore', 'bindings', 'scripts', 'test'),
+
+    # ICU headers are imported.
+    os.path.join('Source', 'JavaScriptCore', 'icu'),
+    os.path.join('Source', 'WebCore', 'icu'),
+    os.path.join('Source', 'WebKit', 'mac', 'icu'),
+    os.path.join('Source', 'WTF', 'icu'),
+    ]
 
 # Extensions of files which are allowed to contain carriage returns.
 _CARRIAGE_RETURN_ALLOWED_FILE_EXTENSIONS = [