Github user JoshRosen commented on the pull request:

    https://github.com/apache/spark/pull/5694#issuecomment-110764427
  
    `pep8` checks still need to be enabled/fixed for this file:
    
    ```
    [joshrosen Spark (1add396...)]$ pep8 dev/run-tests.py
    dev/run-tests.py:34:62: W291 trailing whitespace
    dev/run-tests.py:41:74: W291 trailing whitespace
    dev/run-tests.py:43:1: W293 blank line contains whitespace
    dev/run-tests.py:45:53: W291 trailing whitespace
    dev/run-tests.py:93:20: E203 whitespace before ':'
    dev/run-tests.py:94:20: E203 whitespace before ':'
    dev/run-tests.py:95:20: E203 whitespace before ':'
    dev/run-tests.py:96:20: E203 whitespace before ':'
    dev/run-tests.py:103:62: W291 trailing whitespace
    dev/run-tests.py:104:22: W503 line break before binary operator
    dev/run-tests.py:108:22: W503 line break before binary operator
    dev/run-tests.py:151:65: W291 trailing whitespace
    dev/run-tests.py:153:48: E261 at least two spaces before inline comment
    dev/run-tests.py:154:57: E261 at least two spaces before inline comment
    dev/run-tests.py:155:45: E261 at least two spaces before inline comment
    dev/run-tests.py:157:44: W291 trailing whitespace
    dev/run-tests.py:197:45: W291 trailing whitespace
    dev/run-tests.py:198:50: W291 trailing whitespace
    dev/run-tests.py:199:59: W291 trailing whitespace
    dev/run-tests.py:268:24: W291 trailing whitespace
    dev/run-tests.py:283:60: W291 trailing whitespace
    dev/run-tests.py:288:33: W291 trailing whitespace
    dev/run-tests.py:289:43: W291 trailing whitespace
    dev/run-tests.py:296:57: W291 trailing whitespace
    dev/run-tests.py:317:40: W291 trailing whitespace
    dev/run-tests.py:320:79: W291 trailing whitespace
    dev/run-tests.py:332:44: W291 trailing whitespace
    dev/run-tests.py:342:44: W291 trailing whitespace
    dev/run-tests.py:353:1: W293 blank line contains whitespace
    dev/run-tests.py:392:79: W291 trailing whitespace
    dev/run-tests.py:404:1: W293 blank line contains whitespace
    ```


---
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.
---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to