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

dependency cleanup with grails-bom #384

Merged
merged 1 commit into from
Oct 25, 2024

Conversation

jamesfredley
Copy link
Contributor

No description provided.

hibernate5Version=5.6.15.Final
servletApiVersion=6.0.0
groovyVersion=4.0.23
# Tests are built for this version
h2Version=1.4.200

Choose a reason for hiding this comment

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

h2 is part of the bom.

Copy link
Contributor

Choose a reason for hiding this comment

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

I think the h2 version is held back on purpose here, considering the comment above.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It is, but the tests are based on this version so it is overridden.

Choose a reason for hiding this comment

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

how are tests based on a version of a database?

Copy link
Contributor

Choose a reason for hiding this comment

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

There are compatibility issues with h2:2.x. #385

Choose a reason for hiding this comment

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

well, hopefully we will get to updating them at some point.. My h2 issues always have been pretty straightforward upgraades with the exception of cases where I had to do a persisted database update.

Copy link
Contributor

Choose a reason for hiding this comment

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

I have updated the tests in for h2:2.x but I'll wait for this PR to merge before creating a PR because there are some changes here I need.

hibernate5Version=5.6.15.Final
servletApiVersion=6.0.0
groovyVersion=4.0.23
# Tests are built for this version
h2Version=1.4.200
Copy link
Contributor

Choose a reason for hiding this comment

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

I think the h2 version is held back on purpose here, considering the comment above.

@jamesfredley jamesfredley merged commit 77f66ab into 6.0.x Oct 25, 2024
4 checks passed
@jamesfredley jamesfredley deleted the dependency-cleanup-grails-bom branch October 25, 2024 17:59
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