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

added jakarta support #859

Merged
merged 12 commits into from
Mar 5, 2024
Merged

added jakarta support #859

merged 12 commits into from
Mar 5, 2024

Conversation

RuslanHryn
Copy link
Contributor

@RuslanHryn RuslanHryn commented Mar 4, 2024

Issue link

No issue

Purpose

Support Jakarta to use Dozer in the latest version of Spring Boot and Java EE

Approach

Migrated dependencies and packages

Notes

  • Minumum supported version of JDK is 11 because of new Jakarta dependencies
  • Was not added support of Spring 6
  • Excluded dozer-wildfly-tests module because it is not straightforward to migrate to a new version of wildly with support of Java EE 9 and JDK 11

Open Questions and Pre-Merge TODOs

  • Issue created
  • Unit tests pass
  • Documentation updated
  • Travis build passed

@@ -277,52 +284,10 @@

<profiles>
<profile>
<id>jdk8-build</id>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

now minimum supported version of jdk is 11 because of jakarta dependencies

@@ -88,4 +81,16 @@ public List<Class<RuntimeException>> build(BeanContainer beanContainer) {
public ConfigurationDefinition end() {
return parent;
}

Copy link
Contributor Author

@RuslanHryn RuslanHryn Mar 4, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Unfortunately, lombok has stopped working with the new version of jaxb2-maven-plugin. The reason is unclear.
may be related to mojohaus/jaxb2-maven-plugin#181

@@ -37,8 +37,8 @@
<!--
<module>dozer-osgi-tests-model</module>
<module>dozer-osgi-tests</module>
-->
<module>dozer-wildfly-tests</module>
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

excluded this module because it is straightforward to migrate to new version of wildfly to support jdk 11 and Java EE 9

@garethahealy
Copy link
Collaborator

@RuslanHryn ; any more changes?

@RuslanHryn
Copy link
Contributor Author

@RuslanHryn ; any more changes?

no, that's all

@garethahealy garethahealy changed the title Jakarta support added jakarta support Mar 5, 2024
@garethahealy garethahealy merged commit 326c21d into DozerMapper:master Mar 5, 2024
2 checks 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.

2 participants