[ 
http://jira.codehaus.org/browse/MOJO-1681?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Christopher Hunt closed MOJO-1681.
----------------------------------

    Resolution: Fixed

Patch applied

> [jslint-plugin] improve readability of the code
> -----------------------------------------------
>
>                 Key: MOJO-1681
>                 URL: http://jira.codehaus.org/browse/MOJO-1681
>             Project: Mojo
>          Issue Type: Task
>            Reporter: Stefan Birkner
>            Priority: Minor
>         Attachments: readability.patch
>
>
> There are some issues, which makes it difficult to read the code.
> * The enum Scope is only clutter.
> * There's one big method doExecute, which does all the work.
> I removed the enum and renamed doExecute to checkFolder. According to the 
> guidelines of clean code, checkFolder has been splitted into 
> checkFolderForIssues, reportIssues and optionallyFailForIssues.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://jira.codehaus.org/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

---------------------------------------------------------------------
To unsubscribe from this list, please visit:

    http://xircles.codehaus.org/manage_email


Reply via email to