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

    https://github.com/apache/spark/pull/20338#discussion_r162804771
  
    --- Diff: dev/tox.ini ---
    @@ -13,7 +13,7 @@
     # See the License for the specific language governing permissions and
     # limitations under the License.
     
    -[pep8]
    -ignore=E402,E731,E241,W503,E226
    +[pycodestyle]
    +ignore=E402,E731,E241,W503,E226,E722,E741,E305
    --- End diff --
    
    So, do those `E722,E741,E305` include rules to validate the blank line 
before starting doctests, which is described in the JIRA?
    
    Seems:
    E722: "do not use bare except'"
    E741: "ambiguous variable name 'l'", 
    E305: "expected 2 blank lines after class or function definition"
    
    



---

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

Reply via email to