-
Notifications
You must be signed in to change notification settings - Fork 10
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(deps): Update to Grails 7, Gradle 8.10.2 and grails-core dependency versions #386
Conversation
…nger set by grails-bom
# Conflicts: # gradle.properties # grails-forge-core/src/main/resources/pom.xml # grails-forge-core/src/test/groovy/org/grails/forge/build/gradle/GradleSpec.groovy # grails-forge-core/src/test/groovy/org/grails/forge/feature/grails/GrailsGradlePluginSpec.groovy
Mostly removing micronaut dependencies from generated `build.gradle` I just used comments here pending we figure out the correct course of action.
…y handling - Explicitly set the version for `micronaut-http-client`, previously managed by `grails-bom` in Grails 6. - Remove default inclusion of `micronaut-http-client` as a test dependency when the `micronaut-http-client` feature is not selected.
// micronaut-http-client no longer provides the jackson implementation | ||
generatorContext.addDependency(Dependency.builder() | ||
.groupId("io.micronaut.serde") | ||
.lookupArtifactId("micronaut-serde-jackson") | ||
.compile()); |
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.
Could this be .runtime()
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.
I am going to leave it for now, since I am not 100% sure runtime will cover all standard use cases and since this is an optional feature.
This PR does not include Micronaut or Groovy upgrades.