[CRIU] [PATCH] [v2] zdtm: use flake8 to verify zdtm.py
Tycho Andersen
tycho.andersen at canonical.com
Fri Jun 3 08:06:51 PDT 2016
Hey Andrey,
On Fri, Jun 03, 2016 at 05:33:48AM +0300, Andrey Vagin wrote:
> From: Andrew Vagin <avagin at virtuozzo.com>
>
> and fix various warnings. For example, we mix tab and space indentations.
>
> v2: add flake8.cfg
>
> Cc: Tycho Andersen <tycho.andersen at canonical.com>
Acked-by: Tycho Andersen <tycho.andersen at canonical.com>
This looks great to me, thanks for doing it. My only other suggestion
would be to add it to travis, so that builds fail. Something like,
>From 17bc6d48113ac3b4f5505f400bea9cb886652fc2 Mon Sep 17 00:00:00 2001
From: Tycho Andersen <tycho.andersen at canonical.com>
Date: Fri, 3 Jun 2016 09:05:46 -0600
Subject: [PATCH] travis: run `make lint` to catch pep8 violations in zdtm.py
Signed-off-by: Tycho Andersen <tycho.andersen at canonical.com>
---
scripts/travis-tests | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/scripts/travis-tests b/scripts/travis-tests
index fc7d050..b8fd7c0 100755
--- a/scripts/travis-tests
+++ b/scripts/travis-tests
@@ -4,8 +4,9 @@ set -x -e
test "$1" = "x86_64" || exit 0
apt-get update -qq
-apt-get install -qq protobuf-c-compiler libprotobuf-c0-dev libaio-dev libprotobuf-dev protobuf-compiler python-ipaddr libcap-dev libnl-3-dev
+apt-get install -qq protobuf-c-compiler libprotobuf-c0-dev libaio-dev libprotobuf-dev protobuf-compiler python-ipaddr libcap-dev libnl-3-dev flake8
chmod a+x $HOME
make
+make lint
make -C test/zdtm
python test/zdtm.py run -a -f h,ns -x 'cgroup*'
--
2.7.4
More information about the CRIU
mailing list