Skip to content

Commit 4fef28f

Browse files
committed
spotlessApply
1 parent 63c7186 commit 4fef28f

File tree

5 files changed

+36
-40
lines changed

5 files changed

+36
-40
lines changed

lib/src/compatDiktat1Dot2Dot5/java/com/diffplug/spotless/glue/diktat/compat/DiktatCompat1Dot2Dot5Adapter.java

Lines changed: 21 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -15,24 +15,23 @@
1515
*/
1616
package com.diffplug.spotless.glue.diktat.compat;
1717

18+
import java.io.File;
19+
import java.util.ArrayList;
20+
import java.util.Collections;
21+
import java.util.List;
22+
23+
import javax.annotation.Nullable;
24+
25+
import org.cqfn.diktat.ruleset.rules.DiktatRuleSetProvider;
26+
1827
import com.pinterest.ktlint.core.KtLint;
1928
import com.pinterest.ktlint.core.LintError;
2029
import com.pinterest.ktlint.core.RuleSet;
21-
2230
import com.pinterest.ktlint.core.api.EditorConfigOverride;
2331

2432
import kotlin.Unit;
2533
import kotlin.jvm.functions.Function2;
2634

27-
import org.cqfn.diktat.ruleset.rules.DiktatRuleSetProvider;
28-
29-
import javax.annotation.Nullable;
30-
31-
import java.io.File;
32-
import java.util.ArrayList;
33-
import java.util.Collections;
34-
import java.util.List;
35-
3635
public class DiktatCompat1Dot2Dot5Adapter implements DiktatCompatAdapter {
3736
private final List<RuleSet> ruleSets;
3837
private final Function2<? super LintError, ? super Boolean, Unit> formatterCallback;
@@ -66,18 +65,18 @@ public Unit invoke(LintError lintError, Boolean corrected) {
6665
public String format(final File file, final String content, final boolean isScript) {
6766
errors.clear();
6867
String result = KtLint.INSTANCE.format(new KtLint.ExperimentalParams(
69-
// Unlike Ktlint, Diktat requires full path to the file.
70-
// See https://github.com/diffplug/spotless/issues/1189, https://github.com/analysis-dev/diktat/issues/1202
71-
file.getAbsolutePath(),
72-
content,
73-
ruleSets,
74-
Collections.emptyMap(),
75-
formatterCallback,
76-
isScript,
77-
null,
78-
false,
79-
new EditorConfigOverride(),
80-
false));
68+
// Unlike Ktlint, Diktat requires full path to the file.
69+
// See https://github.com/diffplug/spotless/issues/1189, https://github.com/analysis-dev/diktat/issues/1202
70+
file.getAbsolutePath(),
71+
content,
72+
ruleSets,
73+
Collections.emptyMap(),
74+
formatterCallback,
75+
isScript,
76+
null,
77+
false,
78+
new EditorConfigOverride(),
79+
false));
8180

8281
DiktatReporting.reportIfRequired(errors, LintError::getLine, LintError::getCol, LintError::getDetail);
8382

lib/src/compatDiktat2Dot0Dot0/java/com/diffplug/spotless/glue/diktat/compat/DiktatCompat2Dot0Dot0Adapter.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,9 @@
2323
import java.util.Collections;
2424
import java.util.List;
2525

26+
import org.jetbrains.annotations.NotNull;
27+
import org.jetbrains.annotations.Nullable;
28+
2629
import com.saveourtool.diktat.DiktatFactoriesKt;
2730
import com.saveourtool.diktat.DiktatProcessor;
2831
import com.saveourtool.diktat.api.DiktatCallback;
@@ -32,9 +35,6 @@
3235

3336
import kotlin.Unit;
3437

35-
import org.jetbrains.annotations.NotNull;
36-
import org.jetbrains.annotations.Nullable;
37-
3838
public class DiktatCompat2Dot0Dot0Adapter implements DiktatCompatAdapter {
3939
private final DiktatProcessor processor;
4040
private final DiktatCallback formatterCallback;

lib/src/compatDiktatApi/java/com/diffplug/spotless/glue/diktat/compat/DiktatReporting.java

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -26,11 +26,11 @@ static <T> void reportIfRequired(List<T> errors, ToIntFunction<T> lineGetter, To
2626
error.append("There are ").append(errors.size()).append(" unfixed errors:");
2727
for (T er : errors) {
2828
error.append(System.lineSeparator())
29-
.append("Error on line: ").append(lineGetter.applyAsInt(er))
30-
.append(", column: ").append(columnGetter.applyAsInt(er))
31-
.append(" cannot be fixed automatically")
32-
.append(System.lineSeparator())
33-
.append(detailGetter.apply(er));
29+
.append("Error on line: ").append(lineGetter.applyAsInt(er))
30+
.append(", column: ").append(columnGetter.applyAsInt(er))
31+
.append(" cannot be fixed automatically")
32+
.append(System.lineSeparator())
33+
.append(detailGetter.apply(er));
3434
}
3535
throw new AssertionError(error);
3636
}

lib/src/diktat/java/com/diffplug/spotless/glue/diktat/DiktatFormatterFunc.java

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -17,21 +17,19 @@
1717

1818
import java.io.File;
1919

20+
import com.diffplug.spotless.FormatterFunc;
2021
import com.diffplug.spotless.glue.diktat.compat.DiktatCompat1Dot2Dot5Adapter;
2122
import com.diffplug.spotless.glue.diktat.compat.DiktatCompat2Dot0Dot0Adapter;
2223
import com.diffplug.spotless.glue.diktat.compat.DiktatCompatAdapter;
2324

24-
import com.diffplug.spotless.FormatterFunc;
25-
2625
public class DiktatFormatterFunc implements FormatterFunc.NeedsFile {
2726
private final DiktatCompatAdapter adapter;
2827
private final boolean isScript;
2928

3029
public DiktatFormatterFunc(
31-
String version,
32-
File configFile,
33-
boolean isScript
34-
) {
30+
String version,
31+
File configFile,
32+
boolean isScript) {
3533
if (version.startsWith("1.")) {
3634
this.adapter = new DiktatCompat1Dot2Dot5Adapter(configFile);
3735
} else {

lib/src/main/java/com/diffplug/spotless/kotlin/DiktatStep.java

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -97,10 +97,9 @@ FormatterFunc createFormat() throws Exception {
9797
final File configFile = (config != null) ? config.getOnlyFile() : null;
9898
Class<?> formatterFunc = jar.getClassLoader().loadClass("com.diffplug.spotless.glue.diktat.DiktatFormatterFunc");
9999
Constructor<?> constructor = formatterFunc.getConstructor(
100-
String.class,
101-
File.class,
102-
boolean.class
103-
);
100+
String.class,
101+
File.class,
102+
boolean.class);
104103
return (FormatterFunc.NeedsFile) constructor.newInstance(versionDiktat, configFile, isScript);
105104
}
106105
}

0 commit comments

Comments
 (0)