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

Michael Vorburger.ch commented on FINERACT-702:
-----------------------------------------------

> Can i go ahead and remove this since it is to be replaced by Spotbugs 
> instead. 

sure! May I offer you a Secret Tip, even though you didn't ask for one? ;)  In 
Open Source (and perhaps more generally in life?), do not ask, but just do!

> Integrate SpotBugs and fail build if equals but not hashCode is implemented
> ---------------------------------------------------------------------------
>
>                 Key: FINERACT-702
>                 URL: https://issues.apache.org/jira/browse/FINERACT-702
>             Project: Apache Fineract
>          Issue Type: Improvement
>            Reporter: Michael Vorburger.ch
>            Priority: Major
>              Labels: starter
>
> This project should activate SpotBugs (the successor to FindBugs) and make it 
> fail the build for any violations to a at least a subset of rules which are 
> almost always guaranteed to be bugs.
> For example, 
> https://spotbugs.readthedocs.io/en/stable/bugDescriptions.html#he-class-defines-equals-and-uses-object-hashcode-he-equals-use-hashcode
>  which would have caught what I manually spotted (but shouldn't have to) in 
> https://github.com/apache/fineract/pull/524.
> FINERACT-402 has some history, but it's clearly not activated to enforce.
> FINERACT-696 could then be built on top of this.
> [~myrle] perhaps you know of any volunteer interested in doing this?



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to