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/24 21:59:22 UTC

[GitHub] [fineract] thesmallstar edited a comment on pull request #931: FINERACT-821 Added and enforced NeedBraces Checkstyle

thesmallstar edited a comment on pull request #931:
URL: https://github.com/apache/fineract/pull/931#issuecomment-633306819


   @vorburger I tried to automate this with every plugin/ide that I knew. The best help I could get was a quick fix bulb icon in VSC, which on visiting such if/for loop would help me identify/add braces, but still I had to visit each one of them individually(not a very exciting job tbh). A detailed review will be better? although unlikely if there is a  problem, we might not even notice it after this gets merged. Also, It would be great if we could merge this is 3/4 days to avoid multiple merge conflicts :) 
   


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