check-code: check for repo in revlog and ui in util
authortimeless <timeless@mozdev.org>
Sun, 01 May 2011 16:11:42 +0200
changeset 14137 83a94c2fe6f4
parent 14136 eebf196a8bbe
child 14138 c18204fd35b0
check-code: check for repo in revlog and ui in util
contrib/check-code.py
--- a/contrib/check-code.py	Sun May 01 16:10:51 2011 +0200
+++ b/contrib/check-code.py	Sun May 01 16:11:42 2011 +0200
@@ -208,11 +208,31 @@
     (r'(\()([^)]+\))', repcallspaces),
 ]
 
+inutilpats = [
+  [
+    (r'\bui\.', "don't use ui in util"),
+  ],
+  # warnings
+  []
+]
+
+inrevlogpats = [
+  [
+    (r'\brepo\.', "don't use repo in revlog"),
+  ],
+  # warnings
+  []
+]
+
 checks = [
     ('python', r'.*\.(py|cgi)$', pyfilters, pypats),
     ('test script', r'(.*/)?test-[^.~]*$', testfilters, testpats),
     ('c', r'.*\.c$', cfilters, cpats),
     ('unified test', r'.*\.t$', utestfilters, utestpats),
+    ('layering violation repo in revlog', r'mercurial/revlog\.py', pyfilters,
+     inrevlogpats),
+    ('layering violation ui in util', r'mercurial/util\.py', pyfilters,
+     inutilpats),
 ]
 
 class norepeatlogger(object):