[Reviewers] [Differential] [Commented On] D20: check-code: prohibit `if False` antipattern

krbullock (Kevin Bullock) phabricator-noreply at mercurial-scm.org
Mon Jul 10 19:10:10 UTC 2017


krbullock added a comment.


  Commenting on this one also to test e-mail notifications.

INLINE COMMENTS

> check-code.py:304
>       "use True/False for constant Boolean expression"),
> +    (r'^\s*if False(:| +and)', 'Remove code instead of using `if False`'),
>      (r'(?:(?<!def)\s+|\()hasattr\(',

Cool regex bro

REPOSITORY
  rHG Mercurial

REVISION DETAIL
  https://phab.mercurial-scm.org/D20

EMAIL PREFERENCES
  https://phab.mercurial-scm.org/settings/panel/emailpreferences/

To: durin42, krbullock
Cc: reviewers


More information about the Reviewers mailing list