FINERACT-2734: Upgrade to JDK 25 LTS - #6222
Conversation
Aman-Mittal
left a comment
There was a problem hiding this comment.
LGTM for me @meonkeys @adamsaghy @vidakovic
FYI.
Changes are minimal compared to dependency bump. and also discussed in dev list.
|
There are a few more places under
The 1st should be updated, and the 2nd and 3rd ones should be changed to reference (not duplicate) the first, like we do in the readme. One source of truth FTW. |
meonkeys
left a comment
There was a problem hiding this comment.
There are a few more places under
fineract-doc/to update.
Here's the patch. I can't push directly to the branch so you'll have to apply it yourself (assuming you want it).
Apply it locally with git am. No need to squash.
|
@meonkeys let me try |
|
Should i create a Dedicated Branch for this so that any committer can rebase against develop? |
No, just wait. @IOhacker said he'd apply the patch himself. |
|
@meonkeys @Aman-Mittal for your review |
FINERACT-2734: Upgrade to JDK 25 LTS
Description
FINERACT-2734: Upgrade to JDK 25 LTS
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!
Your assigned reviewer(s) will follow our guidelines for code reviews.