-
Notifications
You must be signed in to change notification settings - Fork 93
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
[Issue_668] Jakarta Data blog post #670
Conversation
f60be93
to
449d4f0
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great, nothing much to say; I just added two comments about minor details.
_posts/2024-10-10-jakarta-data.adoc
Outdated
<plugin> | ||
<groupId>org.apache.maven.plugins</groupId> | ||
<artifactId>maven-compiler-plugin</artifactId> | ||
<version>3.12.0</version> <!-- 3.12 or later is necessary for proper dependency management in annotationProcessorPaths --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Might as well recommend the latest 3.12 while you're at it; 3.12.0 had some rather annoying issues that did impact other projects: https://issues.apache.org/jira/browse/MCOMPILER-567
<version>3.12.0</version> <!-- 3.12 or later is necessary for proper dependency management in annotationProcessorPaths --> | |
<version>3.12.1</version> <!-- 3.12 or later is necessary for proper dependency management in annotationProcessorPaths --> |
_posts/2024-10-10-jakarta-data.adoc
Outdated
---- | ||
|
||
|
||
WARNING: Some users may have learned to configure Hibernate annotation processing by declaring `org.hibernate.orm:hibernate-jpamodelgen` as a `provided` dependency in their pom. With the Hibernate version used with WildFly, link:https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.7#jpamodelgen[this will likely fail, window=_blank]. Use the `maven-compiler-plugin` configuration approach described above. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Alternative link that does not involve Quarkus: https://docs.jboss.org/hibernate/orm/6.3/migration-guide/migration-guide.html#metamodel-generation
WARNING: Some users may have learned to configure Hibernate annotation processing by declaring `org.hibernate.orm:hibernate-jpamodelgen` as a `provided` dependency in their pom. With the Hibernate version used with WildFly, link:https://github.com/quarkusio/quarkus/wiki/Migration-Guide-3.7#jpamodelgen[this will likely fail, window=_blank]. Use the `maven-compiler-plugin` configuration approach described above. | |
WARNING: Some users may have learned to configure Hibernate annotation processing by declaring `org.hibernate.orm:hibernate-jpamodelgen` as a `provided` dependency in their pom. With the Hibernate version used with WildFly, link:https://docs.jboss.org/hibernate/orm/6.3/migration-guide/migration-guide.html#metamodel-generation[this will likely fail, window=_blank]. Use the `maven-compiler-plugin` configuration approach described above. |
449d4f0
to
00392e4
Compare
Thanks for the quick review @yrodiere! I made your suggested changes. |
Resolves #668