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/06/26 16:13:11 UTC

[GitHub] [fineract] thesmallstar commented on pull request #1079: Added and Enforced AvoidHidingCauseException Checkstyle (FINERACT-942)

thesmallstar commented on pull request #1079:
URL: https://github.com/apache/fineract/pull/1079#issuecomment-650264163


   @vorburger  @xurror  can you please review this?
   I have some points:- 
   1. I used suppresswarning at two places( I thought the current approach is correct ).
   2. Some Predefined exceptions were of the form: 
             throw new PredefinedException(x.messege()); // x was lost exception
   this is converted to
             throw new PredefinedException(x); //x is throwable at the exception accepts it,
   This was a a lot of manual work btw :P 
   
   


----------------------------------------------------------------
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