You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@fineract.apache.org by GitBox <gi...@apache.org> on 2020/01/12 12:39:16 UTC

[GitHub] [fineract] vorburger commented on issue #684: more SpotBugs related / inspired code clean up (see FINERACT-702)

vorburger commented on issue #684:  more SpotBugs related / inspired code clean up (see FINERACT-702) 
URL: https://github.com/apache/fineract/pull/684#issuecomment-573411261
 
 
   > Is this still relevant?
   
   Yes, very much so; this contains more but new SpotBugs related / inspired clean up which I coded on top of your recent work! See #685 re. how this was meant to be reviewed as an addendum to #682 .. now that is in, the diff will look clearer. The Conflicts shown are bogus (I don't really understand why GitHub is that stupid; a simple `git rebase` on the CLI fixes them).
   
   I'm just looking into why `ShareAccountIntegrationTests` (only) fails on this PR. Maybe I'll break it up into smaller ones.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services