From 07265798f1881acf02c2ef28f65ccda635170a57 Mon Sep 17 00:00:00 2001 From: Erwan Velu Date: Mon, 9 Apr 2018 15:00:47 +0200 Subject: [PATCH] ceph-container-flake8: Using project's tox.ini The current code is running flake8 from a container to check every python file changed by a PR. The actual code doesn't import the tox.ini of ceph-container which specify a particular configuration like "max-line-length = 99" This lead to a situation where a dev which runs flake on his dev env have a flake reporting now failures while flake from the CI fails. So this patch is about adding project's tox.ini inside the container to insure flake is using it. --- ceph-container-flake8/build/build | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ceph-container-flake8/build/build b/ceph-container-flake8/build/build index 50991ddb..15532a25 100755 --- a/ceph-container-flake8/build/build +++ b/ceph-container-flake8/build/build @@ -21,7 +21,7 @@ function check(){ while read -r filename; do pushd "$(dirname "$filename")" file=$(basename "$filename") - sudo docker run --rm -v "$(pwd)"/"$file":/"$file" eeacms/flake8 /"$file" + sudo docker run --rm -v "$(pwd)"/tox.ini:/tox.ini -v "$(pwd)"/"$file":/"$file" eeacms/flake8 /"$file" popd done return $? -- 2.39.5