[ 
https://issues.apache.org/jira/browse/HADOOP-12204?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Kengo Seki updated HADOOP-12204:
--------------------------------
    Attachment: HADOOP-12204.HADOOP-12111.00.patch

Attaching a wip patch. Pylint result before this patch applied:

{code}
[sekikn@mobile hadoop]$ pylint --indent-string="  " 
dev-support/releasedocmaker.py

(snip)

Global evaluation
-----------------
Your code has been rated at 2.29/10 (previous run: 2.29/10, +0.00)
{code}

And after:

{code}
[sekikn@mobile hadoop]$ pylint --indent-string="  " 
dev-support/releasedocmaker.py

(snip)

Global evaluation
-----------------
Your code has been rated at 8.99/10 (previous run: 2.29/10, +6.70)
{code}

Remaining warnings are as follows. (I'm tempted to accept these ones...)

{code}
C:124, 0: Line too long (103/100) (line-too-long)
C:375, 0: Line too long (105/100) (line-too-long)
C:499, 0: Line too long (112/100) (line-too-long)
C:513, 0: Line too long (101/100) (line-too-long)
C:  1, 0: Missing module docstring (missing-docstring)
F: 30, 2: Unable to import 'simplejson' (import-error)
C: 55, 0: Missing function docstring (missing-docstring)
C: 58, 0: Missing function docstring (missing-docstring)
C: 70, 0: Missing function docstring (missing-docstring)
C: 82, 0: Missing function docstring (missing-docstring)
C: 92, 0: Missing function docstring (missing-docstring)
C: 98, 0: Missing function docstring (missing-docstring)
C:103, 0: Missing function docstring (missing-docstring)
C:133, 2: Missing method docstring (missing-docstring)
R:115, 0: Too few public methods (1/2) (too-few-public-methods)
R:136, 0: Too few public methods (0/2) (too-few-public-methods)
C:168, 2: Missing method docstring (missing-docstring)
C:171, 2: Missing method docstring (missing-docstring)
C:174, 2: Missing method docstring (missing-docstring)
C:183, 2: Missing method docstring (missing-docstring)
C:190, 2: Missing method docstring (missing-docstring)
C:197, 2: Missing method docstring (missing-docstring)
C:203, 2: Missing method docstring (missing-docstring)
C:206, 2: Missing method docstring (missing-docstring)
C:213, 2: Missing method docstring (missing-docstring)
C:220, 2: Missing method docstring (missing-docstring)
C:240, 2: Missing method docstring (missing-docstring)
C:254, 2: Missing method docstring (missing-docstring)
C:259, 2: Missing method docstring (missing-docstring)
C:264, 2: Missing method docstring (missing-docstring)
C:272, 2: Missing method docstring (missing-docstring)
R:281, 2: Too many local variables (16/15) (too-many-locals)
C:325, 2: Missing method docstring (missing-docstring)
R:278, 0: Too few public methods (1/2) (too-few-public-methods)
C:344, 2: Missing method docstring (missing-docstring)
C:353, 2: Missing method docstring (missing-docstring)
C:358, 2: Missing method docstring (missing-docstring)
C:363, 2: Missing method docstring (missing-docstring)
C:374, 0: Missing function docstring (missing-docstring)
R:374, 0: Too many local variables (33/15) (too-many-locals)
R:374, 0: Too many branches (32/12) (too-many-branches)
R:374, 0: Too many statements (147/50) (too-many-statements)
{code}

> releasedocmaker should pass pylint
> ----------------------------------
>
>                 Key: HADOOP-12204
>                 URL: https://issues.apache.org/jira/browse/HADOOP-12204
>             Project: Hadoop Common
>          Issue Type: Sub-task
>          Components: yetus
>            Reporter: Chris Nauroth
>            Priority: Minor
>         Attachments: HADOOP-12204.HADOOP-12111.00.patch
>
>
> releasedocmaker.py currently uses a mix of 2-space and 4-space indentation.  
> It would be good to make this consistent for readability.



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

Reply via email to