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/05/22 10:40:14 UTC

[GitHub] [fineract] thesmallstar commented on pull request #923: FINERACT-821 Added and Enforced ConstantName checkstyle

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


   I think I need to explain in a better way what we are doing here :P 
    A constant is a static and final field, we are trying to define the regex that all constant names should fall under. If anyone doesn't have an objection we can go with the default format, i.e start with a capital letter, and do not allow underscores. 
   
   
   
   


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