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

Michael Vorburger commented on FINERACT-712:
--------------------------------------------

1. *SpotBugs* is getting done in FINERACT-702.
 
2. *Checkstyle* would be really really useful and great as well; I've created a 
new FINERACT-821 to specifically track that.

3. *Error Prone* is another tool which IMHO will add real value, see 
FINERACT-822 for adding that.

4. *Modernizer* is yet another such tool which would be useful, see 
FINERACT-823.

5. *PMD* would need similar work (re-enable it by undoing 
[https://github.com/apache/fineract/pull/612/files] AND removing ignoreFailures 
= true, else it's pointless).

> Fineract codebase should pass PMD, Findbugs and Checkstyle analysis
> -------------------------------------------------------------------
>
>                 Key: FINERACT-712
>                 URL: https://issues.apache.org/jira/browse/FINERACT-712
>             Project: Apache Fineract
>          Issue Type: Improvement
>            Reporter: Vishwas Babu A J
>            Assignee: Awasum Yannick
>            Priority: Major
>              Labels: gsoc2019
>   Original Estimate: 80h
>  Remaining Estimate: 80h
>
> The fineract codebase has findbugs (now spotbugs) and PMD gradle plugins, 
> however, the ignoreFailures option is enabled on the gradle file. Disabling 
> the same and fixing current errors is crucial for the long term 
> maintainability of the Fineract project.
> Another good to have would be checkstyle analysis.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to