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

fix(groovy): Groovy 4 compatibility #385

Merged
merged 1 commit into from
Mar 9, 2024

Conversation

matrei
Copy link
Contributor

@matrei matrei commented Mar 8, 2024

Fix for tests in grails-core failing with
No signature of method: grails.boot.config.GrailsApplicationPostProcessor.postProcessBeanDefinitionRegistry() when running with Groovy 4.

Fix for tests in grails-core failing with
`No signature of method: grails.boot.config.GrailsApplicationPostProcessor.postProcessBeanDefinitionRegistry()`
when running with Groovy 4.
@sdelamo
Copy link
Contributor

sdelamo commented Mar 8, 2024

I don't think Grails support Groovy 4 yet.

@matrei
Copy link
Contributor Author

matrei commented Mar 8, 2024

@sdelamo 👍
Development of Grails 7 is ongoing and this blocking progress.
This PR does not update anything to Groovy 4, it just makes some tests in grails-core work with when running with Groovy 4/Java 17.

@puneetbehl puneetbehl merged commit c6f0bac into grails:3.2.x Mar 9, 2024
4 checks passed
@matrei matrei deleted the matrei/groovy-4-compat branch March 14, 2024 07:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants