Skip to content

Commit f672214

Browse files
authored
Migrate deprecated buildDir usages (#1809)
2 parents 7631fb0 + f327012 commit f672214

File tree

7 files changed

+13
-12
lines changed

7 files changed

+13
-12
lines changed

plugin-gradle/README.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1243,7 +1243,7 @@ To use a fixed binary, omit the `version` and specify a `pathToExe`:
12431243
spotless {
12441244
format 'rome', {
12451245
target '**/*.js','**/*.ts','**/*.json'
1246-
rome().pathToExe("${project.buildDir.absolutePath}/bin/rome")
1246+
rome().pathToExe("${project.layout.buildDirectory.asFile.get().absolutePath}/bin/rome")
12471247
}
12481248
}
12491249
```

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/FormatExtension.java

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -280,9 +280,9 @@ private final FileCollection parseTargetIsExclude(Object target, boolean isExclu
280280
excludes.add(".gradle");
281281
}
282282
// no build folders (flatInclude means that subproject might not be subfolders, see https://github.com/diffplug/spotless/issues/121)
283-
relativizeIfSubdir(excludes, dir, getProject().getBuildDir());
283+
relativizeIfSubdir(excludes, dir, getProject().getLayout().getBuildDirectory().getAsFile().get());
284284
for (Project subproject : getProject().getSubprojects()) {
285-
relativizeIfSubdir(excludes, dir, subproject.getBuildDir());
285+
relativizeIfSubdir(excludes, dir, subproject.getLayout().getBuildDirectory().getAsFile().get());
286286
}
287287
matchedFiles.exclude(excludes);
288288
}
@@ -606,7 +606,7 @@ public T npmInstallCache(final Object npmInstallCache) {
606606
}
607607

608608
public T npmInstallCache() {
609-
this.npmInstallCache = new File(project.getBuildDir(), SPOTLESS_NPM_INSTALL_CACHE_DEFAULT_NAME);
609+
this.npmInstallCache = new File(project.getLayout().getBuildDirectory().getAsFile().get(), SPOTLESS_NPM_INSTALL_CACHE_DEFAULT_NAME);
610610
replaceStep();
611611
return (T) this;
612612
}
@@ -673,7 +673,7 @@ protected FormatterStep createStep() {
673673
devDependencies,
674674
provisioner(),
675675
project.getProjectDir(),
676-
project.getBuildDir(),
676+
project.getLayout().getBuildDirectory().getAsFile().get(),
677677
npmModulesCacheOrNull(),
678678
new NpmPathResolver(npmFileOrNull(), nodeFileOrNull(), npmrcFileOrNull(), Arrays.asList(project.getProjectDir(), project.getRootDir())),
679679
new com.diffplug.spotless.npm.PrettierConfig(

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/JavascriptExtension.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -107,7 +107,7 @@ public FormatterStep createStep() {
107107
devDependencies,
108108
provisioner(),
109109
project.getProjectDir(),
110-
project.getBuildDir(),
110+
project.getLayout().getBuildDirectory().getAsFile().get(),
111111
npmModulesCacheOrNull(),
112112
new NpmPathResolver(npmFileOrNull(), nodeFileOrNull(), npmrcFileOrNull(), Arrays.asList(project.getProjectDir(), project.getRootDir())),
113113
eslintConfig());

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/RegisterDependenciesTask.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,7 @@ void setup() {
6868
taskService = buildServices.registerIfAbsent("SpotlessTaskService" + compositeBuildSuffix, SpotlessTaskService.class, spec -> {});
6969
usesService(taskService);
7070
getBuildEventsListenerRegistry().onTaskCompletion(taskService);
71-
unitOutput = new File(getProject().getBuildDir(), "tmp/spotless-register-dependencies");
71+
unitOutput = new File(getProject().getLayout().getBuildDirectory().getAsFile().get(), "tmp/spotless-register-dependencies");
7272
}
7373

7474
List<FormatterStep> steps = new ArrayList<>();

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/SpotlessDiagnoseTask.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright 2016 DiffPlug
2+
* Copyright 2016-2023 DiffPlug
33
*
44
* Licensed under the Apache License, Version 2.0 (the "License");
55
* you may not use this file except in compliance with the License.
@@ -43,7 +43,8 @@ public SpotlessTask getSource() {
4343
@SuppressFBWarnings("NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE")
4444
public void performAction() throws IOException {
4545
Path srcRoot = getProject().getProjectDir().toPath();
46-
Path diagnoseRoot = getProject().getBuildDir().toPath().resolve("spotless-diagnose-" + source.formatName());
46+
Path diagnoseRoot = getProject().getLayout().getBuildDirectory().getAsFile().get()
47+
.toPath().resolve("spotless-diagnose-" + source.formatName());
4748
getProject().delete(diagnoseRoot.toFile());
4849
try (Formatter formatter = source.buildFormatter()) {
4950
for (File file : source.target) {

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/SpotlessTask.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -148,7 +148,7 @@ public void setTarget(Iterable<File> target) {
148148
}
149149
}
150150

151-
protected File outputDirectory = new File(getProject().getBuildDir(), "spotless/" + getName());
151+
protected File outputDirectory = new File(getProject().getLayout().getBuildDirectory().getAsFile().get(), "spotless/" + getName());
152152

153153
@OutputDirectory
154154
public File getOutputDirectory() {

plugin-gradle/src/main/java/com/diffplug/gradle/spotless/TypescriptExtension.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -118,7 +118,7 @@ public FormatterStep createStep() {
118118
devDependencies,
119119
provisioner(),
120120
project.getProjectDir(),
121-
project.getBuildDir(),
121+
project.getLayout().getBuildDirectory().getAsFile().get(),
122122
npmModulesCacheOrNull(),
123123
new NpmPathResolver(npmFileOrNull(), nodeFileOrNull(), npmrcFileOrNull(), Arrays.asList(project.getProjectDir(), project.getRootDir())),
124124
typedConfigFile(),
@@ -215,7 +215,7 @@ public FormatterStep createStep() {
215215
devDependencies,
216216
provisioner(),
217217
project.getProjectDir(),
218-
project.getBuildDir(),
218+
project.getLayout().getBuildDirectory().getAsFile().get(),
219219
npmModulesCacheOrNull(),
220220
new NpmPathResolver(npmFileOrNull(), nodeFileOrNull(), npmrcFileOrNull(), Arrays.asList(project.getProjectDir(), project.getRootDir())),
221221
eslintConfig());

0 commit comments

Comments
 (0)