Mercurial > public > mercurial-scm > hg-stable
comparison contrib/check-code.py @ 36202:c38e9248f531
contrib: ban $RANDOM using check-code
Differential Revision: https://phab.mercurial-scm.org/D2237
author | Augie Fackler <augie@google.com> |
---|---|
date | Tue, 13 Feb 2018 18:00:59 -0500 |
parents | e28dedf4ff43 |
children | 35d814fe2f30 |
comparison
equal
deleted
inserted
replaced
36201:068f520754ca | 36202:c38e9248f531 |
---|---|
148 (r'env.*-u', "don't use 'env -u VAR', use 'unset VAR'"), | 148 (r'env.*-u', "don't use 'env -u VAR', use 'unset VAR'"), |
149 (r'cp.* -r ', "don't use 'cp -r', use 'cp -R'"), | 149 (r'cp.* -r ', "don't use 'cp -r', use 'cp -R'"), |
150 (r'grep.* -[ABC]', "don't use grep's context flags"), | 150 (r'grep.* -[ABC]', "don't use grep's context flags"), |
151 (r'find.*-printf', | 151 (r'find.*-printf', |
152 "don't use 'find -printf', it doesn't exist on BSD find(1)"), | 152 "don't use 'find -printf', it doesn't exist on BSD find(1)"), |
153 (r'\$RANDOM ', "don't use bash-only $RANDOM to generate random values"), | |
153 ], | 154 ], |
154 # warnings | 155 # warnings |
155 [ | 156 [ |
156 (r'^function', "don't use 'function', use old style"), | 157 (r'^function', "don't use 'function', use old style"), |
157 (r'^diff.*-\w*N', "don't use 'diff -N'"), | 158 (r'^diff.*-\w*N', "don't use 'diff -N'"), |