Refresh Spring Boot and Couchbase dependencies#148
Open
dex-the-ai wants to merge 1 commit into
Open
Conversation
Author
|
Owner ping for this maintenance PR: @teetangh @jdmuntacb Requesting review from the manifest-listed owner(s) where GitHub does not already show a pending review request. |
There was a problem hiding this comment.
Code Review
This pull request updates several dependency and plugin versions in the pom.xml and modifies the Spring Boot version reference in .env.example. However, the review identifies that most of the proposed versions—including those for Spring Boot, the Couchbase SDK, and various Maven plugins—do not exist in Maven Central and will cause build failures. Additionally, the documentation update refers to a non-existent Spring Boot 4.0 release and should be corrected to a valid version.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
4.0.1to4.0.33.10.0to3.11.1, dotenv-java to3.2.0, maven-compiler-plugin to3.15.0, and surefire/failsafe to3.5.5.env.examplecomment drift so it references the current Spring Boot 4.0+ Couchbase property surfaceVerification
JAVA_HOME=/usr/lib/jvm/java-21-openjdk-arm64 ./mvnw clean test -BJAVA_HOME=/usr/lib/jvm/java-17-openjdk-arm64 ./mvnw clean test -BJAVA_HOME=/usr/lib/jvm/java-17-openjdk-arm64 ./mvnw spring-boot:run -Dspring-boot.run.arguments=--server.port=18080curl -I http://127.0.0.1:18080/curl -s http://127.0.0.1:18080/swagger-ui/index.htmlcurl -s http://127.0.0.1:18080/v3/api-docscurl -s http://127.0.0.1:18080/api/v1/airport/airport_1255curl -s "http://127.0.0.1:18080/api/v1/airline/list?country=France&limit=2&offset=0"Evidence
mainpassed locally on JDK 21:18tests,0failures,0errors18tests,0failures,0errorstravel-sample/redirected to Swagger UI,/v3/api-docsreportedQuickstart in Couchbase with Spring Bootwith8paths, and the airport / airline endpoints returned expected live dataspring-boot:runattempt on the default port hit a local8080port conflict, so the walkthrough was re-run successfully on18080tutorial-maintenance/runs/java-springboot-quickstart/2026-05-05T0145Z-subagent/verification.mdtutorial-maintenance/runs/java-springboot-quickstart/2026-05-05T0145Z-subagent/commands.txttutorial-maintenance/runs/java-springboot-quickstart/2026-05-05T0145Z-subagent/baseline-test.logtutorial-maintenance/runs/java-springboot-quickstart/2026-05-05T0145Z-subagent/updated-test-jdk17.logtutorial-maintenance/runs/java-springboot-quickstart/2026-05-05T0145Z-subagent/app-run-18080.logtutorial-maintenance/runs/java-springboot-quickstart/2026-05-05T0145Z-subagent/walkthrough-requests-3.txtNotes