jdaugherty opened a new pull request, #15605:
URL: https://github.com/apache/grails-core/pull/15605

   One of the critical changes I added in the Hibernate 7 branch was a 
restructuring of the bom.  As part of this, it's now possible to use the bom 
and set the hibernate5 version correctly.  Given the size of the hibernate7 
branch, I decided to split this off.  Upon further investigation, I found 
dependency mismatches from the Spring Boot update so I decided to back port 
this all the way to 7.0.x.  
   
   Note: I had to "duplicate" the grails-bom & grails-hibernate5-bom 
definitions because inheriting doesn't allow Gradle to resolve the same way.  I 
could publish 2 project names from grails-hibernate5-bom but I chose to just 
duplicate instead.  
   
   @matrei @jamesfredley given that these versions are mismatching after the 
recent boot update, I think we have to backport this.  I view this change as a 
bug fix since it makes the bom work like you would expect.  I want to commit to 
7.0.x, and merge all the way up.  Then we can also take advantage of this in 
the graphql & other PRs.  Also, it will reduce the scope of the hibernate 7 PR 
that has to be reviewed.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to