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/09/13 15:24:48 UTC

[GitHub] spark pull request #22231: [SPARK-25238][PYTHON] lint-python: Upgrade pycode...

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

    https://github.com/apache/spark/pull/22231#discussion_r217428438
  
    --- Diff: dev/run-tests-jenkins.py ---
    @@ -116,7 +116,7 @@ def run_tests(tests_timeout):
     
         failure_note_by_errcode = {
             # error to denote run-tests script failures:
    -        1: 'executing the `dev/run-tests` script',  # noqa: W605
    +        1: 'executing the `dev/run-tests` script',
    --- End diff --
    
    I don't think we need to exclude this file... just remove the back-ticks here. They're just part of text, not functional.


---

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