Skip to content

(#55) upgraded to Spring Boot 4.0.6, JDK 17, and zonky 2.8.0#56

Open
Eve953 wants to merge 3 commits into
Mom0aut:masterfrom
Eve953:task/upgrade-spring-boot
Open

(#55) upgraded to Spring Boot 4.0.6, JDK 17, and zonky 2.8.0#56
Eve953 wants to merge 3 commits into
Mom0aut:masterfrom
Eve953:task/upgrade-spring-boot

Conversation

@Eve953
Copy link
Copy Markdown

@Eve953 Eve953 commented Jun 1, 2026

Describe your changes

-Changed Spring Boot to 4.0.6 and target compilation properties to JDK 17.
-Changed RELEASE_21 and '.getFirst()' to be compatible with JDK 17

  • Changed embedded-database-spring-test (Zonky) to 2.8.0 to be compatible with the Spring Boot 4 / Jakarta EE namespace baseline.
  • Changed the test environment's ddl-auto setting to create-drop to handle stricter Hibernate 6 type mappings during the integration tests

Issue ticket number and link

Resolves #55

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it is a core feature, I have added thorough tests.
  • I have made corresponding changes to the documentation
  • I used the given Project Style Guide

Comment thread dynamic-relations/src/main/java/at/drm/processor/RelationProcessor.java Outdated
Comment thread dynamic-relations/src/test/java/at/drm/processor/RelationProcessorTest.java Outdated
Comment thread dynamic-relations/src/test/java/at/drm/util/DynamicRelationsUtilsTest.java Outdated
Comment thread dynamic-relations/src/test/java/at/drm/util/DynamicRelationsUtilsTest.java Outdated
Comment thread dynamic-relations/src/test/java/at/drm/util/DynamicRelationsUtilsTest.java Outdated
Comment thread pom.xml Outdated
@Eve953
Copy link
Copy Markdown
Author

Eve953 commented Jun 1, 2026

@Mom0aut I revised it to match JDK 25 now, I replaced all syntax as requested.

Context
Testing Modules fail to compile because of incompatibility with JDK 25.

Cause
Implicit annotation processing is restricted in newer JDK versions. The compilation plugin wasn't explicitly passing the core dynamic-relations artifact as an annotation processor path for the downstream modules' test scopes.

Solution
I updated the root pom.xml under maven-compiler-plugin to explicitly include the project's own processor artifact alongside Lombok inside , along with testing dependencies that needed modification (like updating to RELEASE_25)

@Eve953
Copy link
Copy Markdown
Author

Eve953 commented Jun 2, 2026

@Mom0aut the CI build failed bc the workflow is currently running on JDK 17. This update requires compiling with JDK 25 due to changes in how implicit annotation processing is handled for multi-module structures in modern Java versions. It builds and passes all 16 tests successfully on a JDK 25 environment locally.

@Mom0aut
Copy link
Copy Markdown
Owner

Mom0aut commented Jun 2, 2026

Hey @Eve953 i think you can just update the lines here for the workflow?
https://github.com/Mom0aut/DynamicRelations/blob/master/.github/workflows/maven-pr.yml#L22
and
https://github.com/Mom0aut/DynamicRelations/blob/master/.github/workflows/maven.yml#L21
then the action should run with the jdk 25 can you update them within your pr.

If not tell me i would update them on the main branch and you need to pull them over at your branch.

Thanks for your work 😄

@Eve953
Copy link
Copy Markdown
Author

Eve953 commented Jun 3, 2026

@Mom0aut I updated the yml files everything should be good to go :D

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.

Update to latest Spring Boot, JDK version, and related dependencies

2 participants