Skip to content

Conversation

@devin-ai-integration
Copy link

Summary

This PR migrates the Springboot-BankApp from Java 17 to Java 21. The changes update the Java version configuration across three files:

  • pom.xml: Updated java.version property to 21 and maven-compiler-plugin source/target from 1.8 to 21
  • Dockerfile: Updated both build stage (maven:3.8.3-openjdk-21) and runtime stage (openjdk:21-alpine) base images
  • README.md: Updated Jenkins installation instructions to use openjdk-21-jre

Review & Testing Checklist for Human

  • Verify that maven:3.8.3-openjdk-21 Docker image exists on Docker Hub (I did not verify this)
  • Verify that openjdk:21-alpine Docker image exists on Docker Hub (I did not verify this)
  • Run docker build . locally to confirm the Dockerfile builds successfully
  • Run mvn clean install with Java 21 to verify the application compiles
  • Start the application and verify it runs correctly with Java 21

Notes

Requested by: Joao Esteves (@joao-cognition)

Link to Devin run: https://app.devin.ai/sessions/e2f7ba45250843189654494cbde899fe

- Update java.version property in pom.xml from 17 to 21
- Update maven-compiler-plugin source/target from 1.8 to 21
- Update Dockerfile build stage from maven:3.8.3-openjdk-17 to maven:3.8.3-openjdk-21
- Update Dockerfile runtime stage from openjdk:17-alpine to openjdk:21-alpine
- Update README.md Jenkins installation instructions to use openjdk-21-jre

Co-Authored-By: Joao Esteves <joao.esteves@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

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