Skip to content

Commit 7a7f266

Browse files
dnestorofniephaus
authored andcommitted
Move matrix fill to else branch
1 parent 2fb58fe commit 7a7f266

File tree

1 file changed

+9
-8
lines changed

1 file changed

+9
-8
lines changed

tests/tck-build-logic/src/main/groovy/org.graalvm.internal.tck-harness.gradle

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -120,28 +120,29 @@ Provider<Task> generateMatrixDiffCoordinates = tasks.register("generateMatrixDif
120120
if (!project.hasProperty("baseCommit")) {
121121
throw new GradleException("Missing 'baseCommit' property! Rerun Gradle with '-PbaseCommit=<commit-hash>'")
122122
}
123+
123124
if (diffCoordinates.isEmpty()) {
124125
def matrix = [
125126
"coordinates": ["No matches found!"]
126127
]
127128

128129
matrix.putAll(matrixDefault)
129-
/**
130-
* when we are introducing a new metadata, we should test it against all versions,
131-
* not just the oldest and the newest one (which we are testing by default)
132-
*/
133-
matrix.version.add("21");
134-
matrix.version.add("22");
135-
matrix.version.add("dev");
136-
137130
println "No changed coordinates were found!"
138131
println "::set-output name=matrix::${JsonOutput.toJson(matrix)}"
139132
println "::set-output name=none-found::true"
140133
} else {
141134
def matrix = [
142135
"coordinates": diffCoordinates
143136
]
137+
144138
matrix.putAll(matrixDefault)
139+
/**
140+
* when we are introducing a new metadata, we should test it against all versions,
141+
* not just the oldest and the newest one (which we are testing by default)
142+
*/
143+
matrix.version.add("21")
144+
matrix.version.add("22")
145+
matrix.version.add("dev")
145146
println "::set-output name=matrix::${JsonOutput.toJson(matrix)}"
146147
println "::set-output name=none-found::false"
147148
}

0 commit comments

Comments
 (0)