Github user ashashwat commented on a diff in the pull request:
https://github.com/apache/spark/pull/20436#discussion_r164749320
--- Diff: dev/lint-python ---
@@ -60,9 +60,9 @@ export "PYLINT_HOME=$PYTHONPATH"
export "PATH=$PYTHONPATH:$PATH"
# There is no need to write this output to a file
-#+ first, but we do so so that the check status can
-#+ be output before the report, like with the
-#+ scalastyle and RAT checks.
--- End diff --
Is that so? We have 100 character limit on a single line according to the
style guide. Maybe all 4 lines could be rearranged?
---
---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
