Github user asfgit closed the pull request at:
https://github.com/apache/spark/pull/22266
---
-
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214494032
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214413523
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214413219
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user holdenk commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214412844
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214203205
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user cclauss commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214201610
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214200375
--- Diff: dev/lint-python ---
@@ -82,6 +82,25 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+# stop the build if there are Python syntax
Github user cclauss commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214196909
--- Diff: dev/lint-python ---
@@ -82,6 +82,26 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+python -m pip install flake8
--- End
Github user shaneknapp commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214117901
--- Diff: dev/lint-python ---
@@ -82,6 +82,26 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+python -m pip install flake8
--- End
Github user shaneknapp commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r214114395
--- Diff: dev/lint-python ---
@@ -82,6 +82,26 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+python -m pip install flake8
--- End
Github user HyukjinKwon commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r213880767
--- Diff: dev/lint-python ---
@@ -82,6 +82,26 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+python -m pip install flake8
--- End
Github user viirya commented on a diff in the pull request:
https://github.com/apache/spark/pull/22266#discussion_r213873372
--- Diff: dev/lint-python ---
@@ -82,6 +82,26 @@ else
rm "$PYCODESTYLE_REPORT_PATH"
fi
+python -m pip install flake8
--- End diff
13 matches
Mail list logo