Github user ueshin commented on a diff in the pull request:
https://github.com/apache/spark/pull/20338#discussion_r162852255
--- Diff: dev/lint-python ---
@@ -35,11 +35,10 @@ compile_status="${PIPESTATUS[0]}"
# Get pep8 at runtime so that we don't rely on it being installed on the
build server.
#+ See: https://github.com/apache/spark/pull/1744#issuecomment-50982162
-#+ TODOs:
-#+ - Download pep8 from PyPI. It's more "official".
-PEP8_VERSION="1.7.0"
+# Updated to latest official version for pep8.pep8 is formally renamed to
pycodestyle.
+PEP8_VERSION="2.3.1"
--- End diff --
Should we also use `pycodestyle` instead of `pep8` for variable names or
script names?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
