Migrate from Java 11 to Java 17#9
Conversation
Co-Authored-By: sachet.agarwal@windsurf.com <sachet.agarwal@windsurf.com>
Original prompt from sachet.agarwal2. Update GitHub Actions WorkflowFile: Update the Java version in the CI/CD pipeline: - name: Set up JDK 17
uses: actions/setup-java@v2
with:
distribution: zulu
java-version: '17'3. Update README.md (Optional but Recommended)File: Update any references to Java 11 to Java 17 to reflect the new requirement. Verification StepsAfter making the changes, verify the migration by running:
The existing test suite includes comprehensive coverage across:
All these tests should pass after the migration. DependenciesNo dependency updates are required as the current versions are compatible with Java 17:
NotesThis is a low-risk migration as the codebase uses standard Java features without Java 11-specific APIs, and all dependencies alread... (114 chars truncated...) |
🤖 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:
|
Co-Authored-By: sachet.agarwal@windsurf.com <sachet.agarwal@windsurf.com>
Summary
Migrates the Spring Boot RealWorld application from Java 11 to Java 17. This is a configuration-only change affecting:
build.gradle: UpdatedsourceCompatibilityandtargetCompatibilityfrom 11 to 17.github/workflows/gradle.yml: Updated CI to use JDK 17 (Zulu distribution) and upgraded GitHub Actions from v2 to v4README.md: Updated documentation to reflect Java 17 requirementUpdates since last revision
actions/checkout,actions/setup-java,actions/cache) to fix CI failures caused by deprecated action versionsReview & Testing Checklist for Human
./gradlew bootRun) and verify it starts correctly./gradlew clean test)curl http://localhost:8080/tags) to confirm runtime behaviorNotes
CI is now passing with all tests running on Java 17. The dependencies (Spring Boot 2.6.3, MyBatis 2.2.2, Netflix DGS 4.9.21, JJWT 0.11.2) are confirmed compatible with Java 17.
I was unable to verify the changes locally as the development environment has Java 11 installed.
Link to Devin run
Requested by: @SachetCognition