Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jakarta EE 9+ and JDK 17+ Baseline #45

Merged
merged 3 commits into from
Sep 15, 2023

Conversation

degiorgioo
Copy link
Collaborator

This PR is upgrading the spring framework version and in addition, bumps general dependencies in order to be again up to date.

Davide De added 3 commits August 11, 2023 17:03
- Due compatibility issues with the new Spring Framework version 6.x we want to change along. This includes upgrades of JDK, dependencies and the transition from "import javax.*" to "import jakarta.*"
 Reference: https://spring.io/blog/2021/09/02/a-java-17-and-jakarta-ee-9-baseline-for-spring-framework-6
            https://github.com/spring-projects/spring-framework/wiki/What%27s-New-in-Spring-Framework-6.x

- Jakarta dependencies
  Moved dependencies from javax-* to jakarta-* packages. Transitive dependencies using still javax has been upgraded as well. This includes the following libraries:
       - jakarta.validation-api
       - jakarta.persistence
       - jakarta.servlets
       - hibernate-validator
       - spring (uses jakarta from up version 3.x)

 - JDK17
   Upgrading the current version of the JDK in order to move along the spring baseline. This upgrades the JDK from version 8 to version 17. Dependencies for having a clean build have also been upgraded:
       - jacoco
       - maven-war-plugin
       - lombok

 - TestModelClassWithLotsOfIrrelevantAnnotations
   Removing annotation @SafeHtml due to hibernate-validator API changes - Annotation not present in class path anymore.
- delete AngularJS as client example due to the fact that it works with other clients as well
@velinovskav velinovskav self-requested a review September 15, 2023 09:03
@marcelstoer
Copy link
Collaborator

Thanks for this PR! As your moving vom Java 8 to Java 17 I feel you should bump the project version from 1.x to 2.x as this is breaking.

@velinovskav velinovskav merged commit c4b89b5 into netceteragroup:master Sep 15, 2023
@degiorgioo
Copy link
Collaborator Author

Thanks a lot for your efforts too!

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.

3 participants