Skip to content

Commit e4b92c8

Browse files
authored
Merge pull request #2370 from ryanjbaxter/fix-gradle-deprecations
Fix Gradle deprecations to be compatible with Gradle 10
2 parents 9394801 + b8ae954 commit e4b92c8

File tree

4 files changed

+9
-13
lines changed

4 files changed

+9
-13
lines changed

spring-cloud-contract-tools/spring-cloud-contract-gradle-plugin/build.gradle

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@ buildscript {
66
gradlePluginPortal()
77
}
88
dependencies {
9-
classpath 'com.gradle.publish:plugin-publish-plugin:0.14.0'
9+
classpath 'com.gradle.publish:plugin-publish-plugin:1.3.0'
1010
}
1111
}
1212

@@ -35,13 +35,14 @@ group = 'org.springframework.cloud'
3535
repositories {
3636
mavenLocal()
3737
mavenCentral()
38-
maven { url "https://repo.spring.io/snapshot" }
39-
maven { url "https://repo.spring.io/milestone" }
38+
maven { url = "https://repo.spring.io/snapshot" }
39+
maven { url = "https://repo.spring.io/milestone" }
4040
}
4141

4242
tasks.withType(JavaCompile).configureEach {
4343
sourceCompatibility = JavaVersion.VERSION_17
4444
targetCompatibility = JavaVersion.VERSION_17
45+
options.compilerArgs.add('-Xlint:deprecation')
4546
}
4647

4748
//Dependencies in all subprojects - https://solidsoft.wordpress.com/2014/11/13/gradle-tricks-display-dependencies-for-all-subprojects-in-multi-project-build/

spring-cloud-contract-tools/spring-cloud-contract-gradle-plugin/gradle/release.gradle

Lines changed: 2 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -18,11 +18,8 @@ task javadocJar(type: Jar, dependsOn: javadoc) {
1818
from javadoc.destinationDir
1919
}
2020

21-
artifacts {
22-
archives sourcesJar
23-
archives groovydocJar
24-
archives javadocJar
25-
}
21+
// Add jars as dependencies to the assemble task instead of using the archives configuration
22+
assemble.dependsOn sourcesJar, groovydocJar, javadocJar
2623

2724
ext {
2825
resolveRepoName = { Project project ->

spring-cloud-contract-tools/spring-cloud-contract-gradle-plugin/src/main/java/org/springframework/cloud/contract/verifier/plugin/ContractsCopyTask.java

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,6 @@
6161
import org.springframework.cloud.contract.stubrunner.StubRunnerOptionsBuilder;
6262
import org.springframework.cloud.contract.stubrunner.StubRunnerPropertyUtils;
6363
import org.springframework.cloud.contract.stubrunner.spring.StubRunnerProperties;
64-
import org.springframework.lang.Nullable;
6564
import org.springframework.util.StringUtils;
6665

6766
// TODO: Convert to incremental task: https://docs.gradle.org/current/userguide/custom_tasks.html#incremental_tasks
@@ -262,7 +261,7 @@ private DownloadedData downloadContracts() {
262261
return new DownloadedData(contractsSubDirIfPresent(downloadedContracts), inclusionProperties);
263262
}
264263

265-
private void throwExceptionWhenFailOnNoContracts(@Nullable File file, String contractsRepository) {
264+
private void throwExceptionWhenFailOnNoContracts(File file, String contractsRepository) {
266265
if (StringUtils.hasText(contractsRepository)) {
267266
if (getLogger().isDebugEnabled()) {
268267
getLogger().debug(
@@ -521,7 +520,6 @@ private StubRunnerOptions createStubRunnerOptions() {
521520
return options.build();
522521
}
523522

524-
@Nullable
525523
private String discoverGitCommitId(Map<String, String> props) {
526524
String repositoryUrl = contractRepository.getRepositoryUrl().getOrNull();
527525
if (repositoryUrl != null && ScmStubDownloaderBuilder.isProtocolAccepted(repositoryUrl)) {

spring-cloud-contract-tools/spring-cloud-contract-gradle-plugin/src/main/java/org/springframework/cloud/contract/verifier/plugin/SpringCloudContractVerifierGradlePlugin.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,6 @@ private void configureConfigurations() {
228228
contractTestRuntimeOnly.extendsFrom(testRuntimeOnly);
229229

230230
configurations.create(CONTRACT_TEST_GENERATOR_RUNTIME_CLASSPATH_CONFIGURATION_NAME, conf -> {
231-
conf.setVisible(false);
232231
conf.setCanBeResolved(true);
233232
conf.setCanBeConsumed(false);
234233
conf.attributes(attributes -> {
@@ -366,7 +365,8 @@ private void registerStubsJarTask(ContractVerifierExtension extension,
366365
stubsJar.dependsOn(copyContracts);
367366
stubsJar.dependsOn(generateClientStubs);
368367
});
369-
project.artifacts(artifactHandler -> artifactHandler.add("archives", verifierStubsJar));
368+
// Add verifierStubsJar as a dependency to the assemble task instead of using the archives configuration
369+
project.getTasks().named("assemble").configure(task -> task.dependsOn(verifierStubsJar));
370370
}
371371

372372
private TaskProvider<ContractsCopyTask> registerCopyContractsTask(ContractVerifierExtension extension) {

0 commit comments

Comments
 (0)