Skip to content

Added support for gradle 9#310

Merged
Ben-Edwards-cgi merged 5 commits into
masterfrom
gradle-9-support
Jul 3, 2025
Merged

Added support for gradle 9#310
Ben-Edwards-cgi merged 5 commits into
masterfrom
gradle-9-support

Conversation

@Ben-Edwards-cgi
Copy link
Copy Markdown
Collaborator

Added support for Gradle 9 but update depreciated code usage and updating how we register tasks

import org.gradle.api.plugins.quality.CheckstylePlugin;
import org.gradle.api.tasks.TaskAction;
import org.gradle.util.VersionNumber;
import org.gradle.util.internal.VersionNumber;
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Please can we avoid using an internal API and use something like eg. https://mvnrepository.com/artifact/org.apache.maven/maven-artifact

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Updated :)

Comment thread build.gradle
id 'java-gradle-plugin'
id 'groovy'
id 'com.gradle.plugin-publish' version '1.3.1'
id "uk.gov.hmcts.java" version "0.12.65"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Why are we removing this?

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

This was removed due to compatibility issues which I have since resolved so added this back in (and booped it to the latest version)

@Ben-Edwards-cgi Ben-Edwards-cgi requested a review from banderous July 2, 2025 15:30
@Ben-Edwards-cgi Ben-Edwards-cgi enabled auto-merge July 3, 2025 10:24
@Ben-Edwards-cgi Ben-Edwards-cgi merged commit 5d2c2d4 into master Jul 3, 2025
3 checks passed
@Ben-Edwards-cgi Ben-Edwards-cgi deleted the gradle-9-support branch July 3, 2025 10:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants