Skip to content

Comments

Upgrade Java version from 11 to 17#363

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

Upgrade Java version from 11 to 17#363
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1771520772-upgrade-java-17

Conversation

@devin-ai-integration
Copy link

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

Summary

Updates the project's Java version from 11 to 17 across build configuration, CI pipeline, and documentation. No dependency versions were changed — Spring Boot remains at 2.6.3.

Files changed:

  • build.gradlesourceCompatibility and targetCompatibility updated from '11' to '17'
  • .github/workflows/gradle.yml — JDK setup step changed from java-version: '11' to '17'
  • README.md — "You'll need Java 11 installed" → "You'll need Java 17 installed"

All 68 tests passed locally against Java 17.

Review & Testing Checklist for Human

  • Verify CI passes with JDK 17 (local test pass is necessary but not sufficient)
  • Smoke test ./gradlew bootRun with Java 17 and hit http://localhost:8080/tags — Java 17's stricter internal API encapsulation can surface reflection issues at runtime that unit tests don't catch (relevant for Spring Boot 2.6.3 + MyBatis + DGS)

Notes

  • Spring Boot 2.6.3 officially supports Java 17, but no Spring Boot version bump was made. If any --add-opens or illegal-access warnings appear at runtime, a Spring Boot upgrade may be needed as a follow-up.
  • Requested by: @sanch-cognition
  • Link to Devin run

Open with Devin

Co-Authored-By: Sanch Murugesu <sanch.murugesu@cognition.ai>
@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 3 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.

0 participants