You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@yetus.apache.org by "Sean Busbey (JIRA)" <ji...@apache.org> on 2016/08/01 14:58:20 UTC
[jira] [Commented] (YETUS-430) Refactor releasedocmaker.py in
separate modules
[ https://issues.apache.org/jira/browse/YETUS-430?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15402187#comment-15402187 ]
Sean Busbey commented on YETUS-430:
-----------------------------------
Any chance you have time to do a quick review before Kengo commits this, [~andrew.wang]?
> Refactor releasedocmaker.py in separate modules
> -----------------------------------------------
>
> Key: YETUS-430
> URL: https://issues.apache.org/jira/browse/YETUS-430
> Project: Yetus
> Issue Type: Improvement
> Components: Release Doc Maker
> Reporter: Ajay Yadava
> Assignee: Ajay Yadava
> Attachments: YETUS-430.00.patch
>
>
> releasedocmaker.py has reached 993 lines in length. We have set the upper limit on number of lines in a file to 1000 in .pylintrc. New additions to the core functionality will make it trip that line, so I propose to extract utility functions in a separate module. I hope it will improve readability of the script as well.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)