Skip to content

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

Merged
Mom0aut merged 3 commits into
Mom0aut:masterfrom
Eve953:task/upgrade-spring-boot
Jun 3, 2026
Merged

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

Conversation

@Eve953

@Eve953 Eve953 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

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

Eve953 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor Author

@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

Eve953 commented Jun 2, 2026

Copy link
Copy Markdown
Contributor Author

@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

Mom0aut commented Jun 2, 2026

Copy link
Copy Markdown
Owner

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

Eve953 commented Jun 3, 2026

Copy link
Copy Markdown
Contributor Author

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

@Mom0aut

Mom0aut commented Jun 3, 2026

Copy link
Copy Markdown
Owner

@Eve953 looks good --> merged with master

@Mom0aut Mom0aut merged commit 94e3850 into Mom0aut:master Jun 3, 2026
1 check passed
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