Skip to content

Commit c9fc307

Browse files
nullsGoooler
andauthored
Apply suggestions from code review
Co-authored-by: Goooler <[email protected]>
1 parent f943dc1 commit c9fc307

File tree

2 files changed

+13
-9
lines changed

2 files changed

+13
-9
lines changed

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

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -19,8 +19,12 @@
1919
import java.util.function.Function;
2020
import java.util.function.ToIntFunction;
2121

22-
final class DiktatReporting {
23-
static <T> void reportIfRequired(List<T> errors, ToIntFunction<T> lineGetter, ToIntFunction<T> columnGetter, Function<T, String> detailGetter) {
22+
interface DiktatReporting {
23+
static <T> void reportIfRequired(
24+
List<T> errors,
25+
ToIntFunction<T> lineGetter,
26+
ToIntFunction<T> columnGetter,
27+
Function<T, String> detailGetter) {
2428
if (!errors.isEmpty()) {
2529
StringBuilder error = new StringBuilder();
2630
error.append("There are ").append(errors.size()).append(" unfixed errors:");

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

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -36,11 +36,9 @@ private DiktatStep() {}
3636
private static final String PACKAGE_RELOCATED_VERSION = "2.0.0";
3737

3838
private static final String DEFAULT_VERSION = "2.0.0";
39-
static final String NAME = "diktat";
40-
static final String PACKAGE_PRE_2_0_0 = "org.cqfn.diktat";
41-
static final String PACKAGE = "com.saveourtool.diktat";
42-
static final String MAVEN_COORDINATE_PRE_2_0_0 = PACKAGE_PRE_2_0_0 + ":diktat-rules:";
43-
static final String MAVEN_COORDINATE = PACKAGE + ":diktat-runner:";
39+
private static final String NAME = "diktat";
40+
private static final String MAVEN_COORDINATE_PRE_2_0_0 = "org.cqfn.diktat:diktat-rules:";
41+
private static final String MAVEN_COORDINATE = "com.saveourtool.diktat:diktat-runner:";
4442

4543
public static String defaultVersionDiktat() {
4644
return DEFAULT_VERSION;
@@ -81,11 +79,13 @@ static final class State implements Serializable {
8179

8280
State(String versionDiktat, Provisioner provisioner, boolean isScript, @Nullable FileSignature config) throws IOException {
8381
HashSet<String> pkgSet = new HashSet<>();
82+
final String diktatCoordinate;
8483
if (BadSemver.version(versionDiktat) >= BadSemver.version(PACKAGE_RELOCATED_VERSION)) {
85-
pkgSet.add(MAVEN_COORDINATE + versionDiktat);
84+
diktatCoordinate = MAVEN_COORDINATE + versionDiktat;
8685
} else {
87-
pkgSet.add(MAVEN_COORDINATE_PRE_2_0_0 + versionDiktat);
86+
diktatCoordinate = MAVEN_COORDINATE_PRE_2_0_0 + versionDiktat;
8887
}
88+
this.jar = JarState.from(diktatCoordinate, provisioner);
8989

9090
this.jar = JarState.from(pkgSet, provisioner);
9191
this.versionDiktat = versionDiktat;

0 commit comments

Comments
 (0)