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

Grails 7 compatibility #405

Merged
merged 6 commits into from
Aug 14, 2024
Merged

Grails 7 compatibility #405

merged 6 commits into from
Aug 14, 2024

Conversation

matrei
Copy link
Contributor

@matrei matrei commented Jul 27, 2024

This PR updates the project to Java 17, Groovy 4 and Jakarta.
Hopefully the SNAPSHOT can be used for getting some more tests in grails-gsp update to pass.

The example test project does not currently work and blocks
progress. This commit disables it from the build temporarily
until conditions have changed.
@matrei
Copy link
Contributor Author

matrei commented Jul 27, 2024

Related: grails/grails-core#13555

@matrei matrei self-assigned this Aug 1, 2024
build.gradle Outdated
@@ -42,7 +52,7 @@ subprojects { project ->
}

repositories {
mavenLocal()
//mavenLocal()
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
//mavenLocal()

@@ -31,6 +31,6 @@ include 'grails-testing-support',
'grails-web-testing-support',
'grails-gorm-testing-support'

include 'examples-demo33'
project(":examples-demo33").projectDir = new File(settingsDir, "examples/demo33")
//include 'examples-demo33'
Copy link
Contributor

Choose a reason for hiding this comment

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

why comment this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@sdelamo The example test project does not currently work and blocks
progress. This commit disables it from the build temporarily
until conditions have changed.

@sdelamo sdelamo merged commit 61c3802 into 4.0.x Aug 14, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants