This is an automated email from the ASF dual-hosted git repository.

altay pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/beam.git


The following commit(s) were added to refs/heads/master by this push:
     new 1bf2229  [BEAM-3738] Add more flake8 tests to run_pylint.sh
     new 6b357c6  This closes #4798
1bf2229 is described below

commit 1bf22296c9ee2218f393d99063d06b5bb22e0f8d
Author: cclauss <[email protected]>
AuthorDate: Tue Mar 6 21:05:08 2018 +0100

    [BEAM-3738] Add more flake8 tests to run_pylint.sh
---
 sdks/python/run_mini_py3lint.sh | 3 ++-
 sdks/python/run_pylint.sh       | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

diff --git a/sdks/python/run_mini_py3lint.sh b/sdks/python/run_mini_py3lint.sh
index f5e9acd..0fd9ef7 100755
--- a/sdks/python/run_mini_py3lint.sh
+++ b/sdks/python/run_mini_py3lint.sh
@@ -37,4 +37,5 @@ if test $# -gt 0; then
 fi
 
 echo "Running flake8 for module $MODULE:"
-flake8 $MODULE --count --select=E999 --show-source --statistics
+# TODO(BEAM-3959): Add F821 (undefined names) as soon as that test passes
+flake8 $MODULE --count --select=E9,F822,F823 --show-source --statistics
diff --git a/sdks/python/run_pylint.sh b/sdks/python/run_pylint.sh
index e3f9a8e..89c46ce 100755
--- a/sdks/python/run_pylint.sh
+++ b/sdks/python/run_pylint.sh
@@ -63,7 +63,8 @@ pylint -j8 "$MODULE" --ignore-patterns="$FILES_TO_IGNORE"
 echo "Running pycodestyle for module $MODULE:"
 pycodestyle "$MODULE" --exclude="$FILES_TO_IGNORE"
 echo "Running flake8 for module $MODULE:"
-flake8 $MODULE --count --select=E999 --show-source --statistics
+# TODO(BEAM-3959): Add F821 (undefined names) as soon as that test passes
+flake8 $MODULE --count --select=E9,F822,F823 --show-source --statistics
 
 echo "Running isort for module $MODULE:"
 # Skip files where isort is behaving weirdly

-- 
To stop receiving notification emails like this one, please contact
[email protected].

Reply via email to