Skip to content

Update gradle to 8.6 #1535

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

Closed
wants to merge 9 commits into from
Closed

Conversation

theqeeqez
Copy link
Contributor

@theqeeqez theqeeqez commented Feb 10, 2024

Update Gradle to 8.6 and Get rid of Deprecations

What is done

  1. Run ./gradlew wrapper --gradle-version=8.6 to switch to the new gradle
  2. Run ./gradlew build --warning-mode all to check all the deprecations we have
  3. Run Intellij code inspection to check for deprecations used within gradle-related stuff

Fixed Deprecations

  1. Java convention sourceCompatibility should be called as java plugin configuration Read docs on gradle site
  2. .getConvention().getPlugin() now .getExtensions().getByType() in SpringArtifactoryPlugin.java Read docs on gradle site
  3. .getBuildDir() now .getLayout().getBuildDirectory().getAsFile().get() in SpringPublishLocalPlugin.java

Updated dependencies

Older versions use deprecated gradle features or do not support new gradle

  • io.github.gradle-nexus:publish-plugin from 1.1.0 to 2.0.0-rc-2
  • org.jfrog.buildinfo:build-info-extractor-gradle from 4.29.0 to 5.2.0
  • spring-javaformat-gradle-plugin from 0.0.31 to 0.0.41
  • io.spring.nohttp:nohttp-gradle from 0.0.10 to 0.0.11

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Feb 10, 2024
@jgrandja jgrandja self-assigned this Feb 28, 2024
@jgrandja jgrandja added type: enhancement A general enhancement and removed status: waiting-for-triage An issue we've not yet triaged labels Feb 28, 2024
@jgrandja jgrandja added this to the 1.3.0-M3 milestone Feb 28, 2024
@jgrandja jgrandja mentioned this pull request Feb 28, 2024
@jgrandja jgrandja closed this in a05d89d Feb 28, 2024
jgrandja added a commit that referenced this pull request Feb 28, 2024
@jgrandja
Copy link
Collaborator

Thanks for the PR @qeeqez ! This is now merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: enhancement A general enhancement
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants