Add comprehensive Java 24 upgrade plan document#19
Open
dillonvargo wants to merge 1 commit into
Open
Conversation
Co-Authored-By: Dillon Vargo <dillonvargo@gmail.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of what I changed
Adds
JAVA_24_UPGRADE_PLAN.md— a planning document that outlines a phased approach to upgrade OpenMRS Core from Java 21 to Java 24 as the primary target. No code changes are included; this is documentation only.The plan covers 5 phases:
tools.jarprofiles, redundant Mockito profiles, legacy.travis.ymljavax.mail→jakarta.mail(with specific source file changes listed),javax.servlet-apiin test-modulejavaCompilerVersion, Dockerfile JDK args, CI matrix, JVM argumentsEach item includes file locations, exact actions, risk levels, and dependency ordering.
Issue I worked on
N/A — this is a planning document requested ahead of implementation.
Important items for reviewer
jakarta.mail-api:2.1.3withangus-mail:2.0.3, Velocity2.4.1, Groovy4.0.24).javax.mailorjavax.servletwill break. Consider whether a more detailed module migration guide or communication plan is needed.javax.xml.*,javax.crypto.*,javax.imageio.*from migration (they're JDK-internal, not Jakarta EE), but reviewers familiar with the codebase should confirm no otherjavax.*→jakarta.*migrations were missed.Checklist: I completed these to help reviewers :)
mvn clean packageright before creating this pull request and added all formatting changes to my commit.Link to Devin run: https://app.devin.ai/sessions/4fe707b80dc44b53859baf47a63d214e
Requested by: @dillonvargo