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(deps): Update to Grails 7, Gradle 8.10.2 and grails-core dependency versions #386

Merged
merged 9 commits into from
Oct 10, 2024

Conversation

jamesfredley
Copy link
Contributor

@jamesfredley jamesfredley commented Oct 8, 2024

This PR does not include Micronaut or Groovy upgrades.

@jamesfredley jamesfredley self-assigned this Oct 8, 2024
jamesfredley and others added 3 commits October 9, 2024 08:42
# 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.
@jamesfredley jamesfredley changed the title Update to Grails 7, Gradle 8.10.2 and grails-core dependency versions fix(deps): Update to Grails 7, Gradle 8.10.2 and grails-core dependency versions Oct 10, 2024
// micronaut-http-client no longer provides the jackson implementation
generatorContext.addDependency(Dependency.builder()
.groupId("io.micronaut.serde")
.lookupArtifactId("micronaut-serde-jackson")
.compile());
Copy link
Contributor

Choose a reason for hiding this comment

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

Could this be .runtime()

Copy link
Contributor Author

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.

@jamesfredley jamesfredley merged commit ed9e747 into 7.0.x Oct 10, 2024
14 checks passed
@jamesfredley jamesfredley deleted the update-dependencies-grails-7 branch October 10, 2024 14:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants