Skip to content

Comments

Java 17 Migration - Phase 1 & 2: Build Configuration & CI Updates#362

Open
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1771462045-java-17-migration
Open

Java 17 Migration - Phase 1 & 2: Build Configuration & CI Updates#362
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1771462045-java-17-migration

Conversation

@devin-ai-integration
Copy link

@devin-ai-integration devin-ai-integration bot commented Feb 19, 2026

Java 17 Migration — Phase 1 & 2: Build Configuration Updates

Summary

Updates all Java version references from 11 to 17 across build configuration, CI pipeline, and documentation. This is a config-only change as part of a phased migration — no dependencies or Java source code have been modified yet.

Changes:

  • build.gradle: sourceCompatibility / targetCompatibility'17'
  • .github/workflows/gradle.yml: JDK setup step → java-version: '17'
  • README.md: Getting started prerequisite → Java 17

Dependency audit (for future phases):

Dependency Current Version Recommended Upgrade
Spring Boot 2.6.3 2.7.x
MyBatis Spring Boot Starter 2.2.2 2.3.x or 3.0.x
Netflix DGS GraphQL 4.9.21 5.5.x+
Netflix DGS Codegen plugin 5.0.6 needs upgrade
Joda-Time 2.10.13 migrate to java.time API

Review & Testing Checklist for Human

  • Confirm CI failure is expected: dependencies (Spring Boot 2.6.3, DGS 4.9.21, etc.) have not been upgraded yet and may not be fully Java 17 compatible. Verify this phased approach is acceptable before merging.
  • Check for other Java 11 references: search the repo for any remaining hardcoded Java 11 mentions (e.g., Dockerfiles, other CI configs, properties files) that this PR may have missed.
  • Test plan: After merging, proceed with Phase 3 (dependency upgrades) and Phase 4 (code changes) before expecting a green build.

Notes


Open with Devin

Phase 1 & 2 of Java 17 migration:
- Update sourceCompatibility and targetCompatibility to 17 in build.gradle
- Update CI pipeline to use JDK 17 (Zulu distribution)
- Update README.md to reflect Java 17 requirement

Note: Dependencies have not been upgraded yet (Phase 3).
The build may not pass until dependencies are updated.

Co-Authored-By: Vedant Khanna <vedantkhanna@gmail.com>
@devin-ai-integration
Copy link
Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

Copy link
Author

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 2 additional findings.

Open in Devin Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant