Mercurial Mercurial > hg / changeset
summary | shortlog | changelog | graph | tags | bookmarks | branches | files | changeset | raw | zip | gz | bz2 | help
Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
check-code: prohibit `if False` antipattern
authorAugie Fackler <raf@durin42.com>
Fri, 07 Jul 2017 15:11:11 -0400
changeset 33369 d36bcba91845
parent 33368 ffb30661f672
child 33370 482320104672
check-code: prohibit `if False` antipattern Differential Revision: https://phab.mercurial-scm.org/D20
contrib/check-code.py file | annotate | diff | comparison | revisions 
--- a/contrib/check-code.py	Fri Jul 07 15:08:23 2017 -0400
+++ b/contrib/check-code.py	Fri Jul 07 15:11:11 2017 -0400
@@ -301,6 +301,7 @@
      "comparison with singleton, use 'is' or 'is not' instead"),
     (r'^\s*(while|if) [01]:',
      "use True/False for constant Boolean expression"),
+    (r'^\s*if False(:| +and)', 'Remove code instead of using `if False`'),
     (r'(?:(?<!def)\s+|\()hasattr\(',
      'hasattr(foo, bar) is broken on py2, use util.safehasattr(foo, bar) '
      'instead', r'#.*hasattr-py3-only'),
Mercurial