You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@spark.apache.org by srowen <gi...@git.apache.org> on 2018/10/22 19:03:49 UTC

[GitHub] spark pull request #22425: [SPARK-23367][Build] Include python document styl...

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

    https://github.com/apache/spark/pull/22425#discussion_r227001114
  
    --- Diff: dev/lint-python ---
    @@ -99,6 +104,29 @@ else
         echo "flake8 checks passed."
     fi
     
    +# Check python document style, skip check if pydocstyle is not installed.
    --- End diff --
    
    Nit: the indentation below looks different (4-space) from the rest of the script (2-space)


---

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