Skip to content

Upgrade build to Java 11 / Spring Boot 2.7.18 / Gradle 7.6.4 - #80

Open
devin-ai-integration[bot] wants to merge 2 commits into
masterfrom
devin/1788954337-java11-upgrade
Open

devin-ai-integration[bot] wants to merge 2 commits into
masterfrom
devin/1788954337-java11-upgrade

Conversation

@devin-ai-integration

@devin-ai-integration devin-ai-integration Bot commented Sep 9, 2026

Copy link
Copy Markdown

Summary

Java 8 → 11 across both build systems, with the coordinated bumps needed for Java 11 support:

  • pom.xml: java.version 1.8 → 11, spring-boot-starter-parent 2.0.2.RELEASE → 2.7.18. Also <packaging>pom</packaging>jar — with pom packaging Maven never compiled src/main/java at all, so the build "passed" without exercising the sources or producing an artifact.
  • build.gradle: sourceCompatibility/targetCompatibility → 11 plus options.release = 11 (enforces the Java 11 API surface on newer JDKs), spring-boot-gradle-plugin → 2.7.18, compile/testCompileimplementation/testImplementation, bootJar { baseName/version }archiveBaseName/archiveVersion (old properties removed in Gradle 7/8). Added spring-boot-starter-jdbc, h2, and spring-boot-properties-migrator to match the pom — Application.java imports JdbcTemplate, so the Gradle build did not compile without them.
  • gradle/wrapper/gradle-wrapper.properties: Gradle 4.6 → 7.6.4.
  • gradlew/mvnw: made executable. README prerequisite updated to Java 11.

No Java source changes were needed. Verified on OpenJDK 11.0.32: ./gradlew clean build and mvn clean package both succeed and produce gs-spring-boot-0.1.0.jar. (./mvnw itself could not be verified here — its Maven 3.3.9 distribution download from repo1.maven.org was rate-limited with HTTP 429.)

Left untouched: the defunct gturnquist-quoters.cfapps.io call in Application.java (runtime-only, unrelated to the upgrade), and the tracked target/classes/*.class files.

Devin-Org: engineering

Link to Devin session: https://app.devin.ai/sessions/27eb2626eee7463397268bd04addb238
Open in Devin Desktop: https://app.devin.ai/desktop/session/27eb2626eee7463397268bd04addb238?variant=devin
Requested by: @bryce-h-t

Co-Authored-By: Bryce Hunt <bryce@cognition.ai>
@devin-ai-integration

Copy link
Copy Markdown
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, CI, and merge conflict monitoring

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

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Devin Review found 4 potential issues.

Devin Review

Comment thread build.gradle Outdated
Comment thread build.gradle
Comment thread build.gradle Outdated
Comment thread pom.xml
…quisite

Co-Authored-By: Bryce Hunt <bryce@cognition.ai>
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