Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/20338#discussion_r163467168
--- Diff: dev/tox.ini ---
@@ -13,7 +13,7 @@
# See the License for the specific language governing permissions and
# limitations under the License.
-[pep8]
-ignore=E402,E731,E241,W503,E226
+[pycodestyle]
+ignore=E402,E731,E241,W503,E226,E722,E741,E305
--- End diff --
Yea .. there look actually many instances about it. I manually ran it
against this PR. Will maybe take this one and open a PR separately after double
checking each rule. Didn't take a look carefully yet.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
