[ 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