Mercurial > evolve
annotate tests/test-check-flake8.t @ 5917:94aed9c7ce69 stable
next: refactor two if blocks into an if-elif block
We already checked the value of `needevolve and opts['evolve']` in the first if
block above. In the first one it need to be True, in the second it needs to be
False for the statements to execute. Since not(a or b) = not a and not b, we
can join the second if block to the first one with an elif while dropping the
explicit check.
For me, this works better, because we're doing the same thing in both of these
blocks (checking if working copy is dirty or not).
author | Anton Shestakov <av6@dwimlabs.net> |
---|---|
date | Sat, 15 May 2021 20:40:19 +0800 |
parents | 39bc93749027 |
children | 89349d870f6b |
rev | line source |
---|---|
5338
6a50faea786d
tests: use hghaveaddon.py to check for flake8
Anton Shestakov <av6@dwimlabs.net>
parents:
5337
diff
changeset
|
1 #require test-repo flake8 |
1930 | 2 |
3 Copied from Mercurial core (60ee2593a270) | |
4 | |
5 $ cd "`dirname "$TESTDIR"`" | |
6 | |
5649
39bc93749027
tests: update comment in test-check-flake8.t
Anton Shestakov <av6@dwimlabs.net>
parents:
5338
diff
changeset
|
7 run flake8 on all tracked files ending in .py or with a python shebang |
2002 | 8 |
3390
b3dbba6e34c9
evolve: start vendoring third party package cbor
Pulkit Goyal <7895pulkit@gmail.com>
parents:
2121
diff
changeset
|
9 $ hg files -0 'set:(**.py or grep("^#!.*python")) - removed()' \ |
b3dbba6e34c9
evolve: start vendoring third party package cbor
Pulkit Goyal <7895pulkit@gmail.com>
parents:
2121
diff
changeset
|
10 > -X hgext3rd/evolve/thirdparty \ |
b3dbba6e34c9
evolve: start vendoring third party package cbor
Pulkit Goyal <7895pulkit@gmail.com>
parents:
2121
diff
changeset
|
11 > 2>/dev/null \ |
5337
f354de498737
tests: actually use the correct python version for flake8/pyflakes
Anton Shestakov <av6@dwimlabs.net>
parents:
3390
diff
changeset
|
12 > | xargs -0 "$PYTHON" -m flake8 |