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/09 17:56:10 UTC

[GitHub] [fineract] vorburger opened a new pull request #859: make javac fail, not just warn, for (Xlint) unchecked (FINERACT-959)

vorburger opened a new pull request #859:
URL: https://github.com/apache/fineract/pull/859


   This is just a first step - we should aim for -Xlint:all! ;)
   
   see FINERACT-959


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



[GitHub] [fineract] thesmallstar commented on pull request #859: make javac fail, not just warn, for (Xlint) unchecked (FINERACT-959)

Posted by GitBox <gi...@apache.org>.
thesmallstar commented on pull request #859:
URL: https://github.com/apache/fineract/pull/859#issuecomment-629457264


   LGTM!


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



[GitHub] [fineract] vorburger commented on pull request #859: make javac fail, not just warn, for (Xlint) unchecked (FINERACT-959)

Posted by GitBox <gi...@apache.org>.
vorburger commented on pull request #859:
URL: https://github.com/apache/fineract/pull/859#issuecomment-629385059


   @ptuomola @xurror @awasum @thesmallstar does this one LGYT? Merge for me, or OK to self merge?


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



[GitHub] [fineract] awasum merged pull request #859: make javac fail, not just warn, for (Xlint) unchecked (FINERACT-959)

Posted by GitBox <gi...@apache.org>.
awasum merged pull request #859:
URL: https://github.com/apache/fineract/pull/859


   


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



[GitHub] [fineract] ptuomola commented on pull request #859: make javac fail, not just warn, for (Xlint) unchecked (FINERACT-959)

Posted by GitBox <gi...@apache.org>.
ptuomola commented on pull request #859:
URL: https://github.com/apache/fineract/pull/859#issuecomment-629444617


   Looks great to me! 


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