Github user cclauss commented on the issue:

    https://github.com/apache/spark/pull/22231
  
    @holdenk Can you please do me a big favor and create a separate PR that 
makes sure that we are doing at least __flake8 . --count 
--select=E901,E999,F821,F822,F823 --show-source --statistics__ on the whole 
codebase on Python 2 and Python 3?  When I create PRs on this project, I make 
so many mistakes that it takes pages of conversation and weeks of testing to 
get them merged.  I just want to make sure that the project never backslides on 
F821s because it has taken [a lot of 
effort](https://github.com/apache/spark/pulls?q=is%3Apr+author%3Acclauss+is%3Aclosed)
 to get us clean.  Thanks.


---

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

Reply via email to