Github user HyukjinKwon commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22963#discussion_r231463118
  
    --- Diff: dev/lint-python ---
    @@ -87,27 +91,46 @@ 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 --max-line-length=100 
--show-source --statistics
    --- End diff --
    
    BTW, it was never a good idea to use external executable program directly 
without any check.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to