vorburger opened a new pull request #913: URL: https://github.com/apache/fineract/pull/913
We *ARE* using Guava, but, unless we have a real need for a more recent specific version, it seems "safer" (and less hassle to maintain bumping) just using whatever version we transitively inherit from other libraries we use that themselves depend on Guava (currently Swagger, Mustache and activemq-broker). Currently that "whatever version" is a 28.2 (we used 28.1 before). This removes two fairly different kinds of Guava dependencies we have: One for run-time (above), and one for build time which seems to have been added due to some SpotBugs related issue (but it now works without that, so that must have been some issue in SpotBugs that meanwhile go fixed). ---------------------------------------------------------------- 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: [email protected]
