Github user nchammas commented on the pull request:
https://github.com/apache/spark/pull/1505#issuecomment-49698547
If we have to, we could probably somehow package `pep8` and its
dependencies as a standalone. It's doable but I think also a bit ugly and
harder to update.
As for adding to the Jenkins scripts, shall I take a crack at that in a
separate PR?
How are you thinking we approach that? There are at least a couple of
things we could do, like:
* update `/dev/run-tests` to include `pep8` checks directly
* update `/dev/run-tests` to call a `pythonstyle` script under `/dev/`
We should probably update step 4 under ["Contributing
Code"](https://cwiki.apache.org/confluence/display/SPARK/Contributing+to+Spark#ContributingtoSpark-ContributingCode)
in the "Contributing to Spark Guide" to cover Python style as well.
Also, would it make sense to replace `sbt/sbt scalastyle` with a new
`check-style` script in `/dev/` that checks style for both Scala and Python
code?
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
