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

    https://github.com/apache/spark/pull/22425#discussion_r220757773
  
    --- Diff: dev/tox.ini ---
    @@ -14,6 +14,8 @@
     # limitations under the License.
     
     [pycodestyle]
    -ignore=E226,E241,E305,E402,E722,E731,E741,W503,W504
    +ignore=E226,E241,E305,E402,E722,E731,E741,W503,W504,W605
    --- End diff --
    
    The intent of this PR was on pydocstyle changes, and this was pycodestyle 
break, so was unsure if that fix can be included in this PR. I had seen W605 
code style broken in that build, and to make it pass added this. I agree 
completely the fix for W605 can be done with no-PR trivial fix., will check 
again.



---

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

Reply via email to