[ 
https://issues.apache.org/jira/browse/SPARK-12731?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15094540#comment-15094540
 ] 

Joseph K. Bradley commented on SPARK-12731:
-------------------------------------------

If it's hard to enforce with style checks, then we can skip this task.

If we don't want to enforce it with style checks, then I'd say we should update 
the style guide to say it's OK to have long docstring lines.

Otherwise, we should probably incur this 1-time cost.  It's easier for people 
to run style checks than for reviewers to comment on style.

CC: [~rxin] who has worked on style a lot

> PySpark docstring cleanup
> -------------------------
>
>                 Key: SPARK-12731
>                 URL: https://issues.apache.org/jira/browse/SPARK-12731
>             Project: Spark
>          Issue Type: Improvement
>          Components: Documentation, PySpark
>            Reporter: holdenk
>            Priority: Trivial
>
> We don't currently have any automated checks that our PySpark docstring lines 
> are within pep8/275/276 lenght limits (since the pep8 checker doesn't handle 
> this). As such there are ~400 non-comformant docstring lines. This JIRA is to 
> fix those docstring lines and add a command to lint python to fail on long 
> lines.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

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

Reply via email to