Github user rxin commented on the pull request:
https://github.com/apache/spark/pull/1505#issuecomment-49698681
Let's definitely add the pep8 check in a separate PR. I'm waiting for
Jenkins to come back positive before merging this pull request.
I think it'd make sense to have a linter script for scala (dev/lint-scala),
and a linter script (dev/lint-python) for python so it is easier for devs to
run. lint-scala can just call scalastyle, while lint-python calls pep8.
And then run-tests should call both. And yes, let's update the wiki once we
have this checked in.
---
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.
---
