Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214413219
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax errors or undefined names
+flake8 . --count --select=E901,E999,F821,F822,F823 --show-source
--statistics
+flake8_status="${PIPESTATUS[0]}"
+# exit-zero treats all errors as warnings. The GitHub editor is 127 chars
wide
+flake8 . --count --exit-zero --max-complexity=10 --max-line-length=127
--statistics
--- End diff --
From https://spark.apache.org/contributing.html our max line length should
be 100
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
