ptuomola commented on pull request #873:
URL: https://github.com/apache/fineract/pull/873#issuecomment-628383843


   @vorburger Unfortunately I'm not an expert on Guava... but looking at the 
dependencies:
   
   - The only reference in Fineract source to Guava that I was able to find is 
in some comments in org.apache.fineract.infrastructure.bulkimport.populator, 
and everything seems to compile OK even if I remove the Guava dependency from 
dependencies.gradle. Do we even use this in Fineract?
   
   - checkstyle is already expecting 0.29
   - If we upgrade Mustache to latest version (0.9.6), it no longer seems to 
depend on Guava
   
   So after that, it's only Swagger and ActiveMQ that are expecting Guava 27 / 
28 but would get 29. Looking at the changelog I did not see anything that made 
me worried, but of course you never know...
   


----------------------------------------------------------------
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:
us...@infra.apache.org


Reply via email to