Skip to content

Commit 2dc9b31

Browse files
mark-vieiraelasticsearchmachine
andauthored
[9.1] Run transport version validation tasks on precommit (elastic#134482) (elastic#134490)
* Run transport version validation tasks on precommit (elastic#134482) (cherry picked from commit 36181aa) # Conflicts: # build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/transport/TransportVersionResourcesPlugin.java * [CI] Auto commit changes from spotless --------- Co-authored-by: elasticsearchmachine <[email protected]>
1 parent 0078040 commit 2dc9b31

File tree

2 files changed

+8
-2
lines changed

2 files changed

+8
-2
lines changed

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/transport/TransportVersionReferencesPlugin.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
package org.elasticsearch.gradle.internal.transport;
1111

1212
import org.elasticsearch.gradle.internal.ProjectSubscribeServicePlugin;
13+
import org.elasticsearch.gradle.internal.conventions.precommit.PrecommitPlugin;
14+
import org.elasticsearch.gradle.internal.conventions.precommit.PrecommitTaskPlugin;
1315
import org.elasticsearch.gradle.util.GradleUtils;
1416
import org.gradle.api.Plugin;
1517
import org.gradle.api.Project;
@@ -23,6 +25,7 @@ public class TransportVersionReferencesPlugin implements Plugin<Project> {
2325
@Override
2426
public void apply(Project project) {
2527
project.getPluginManager().apply(LifecycleBasePlugin.class);
28+
project.getPluginManager().apply(PrecommitTaskPlugin.class);
2629

2730
project.getPlugins()
2831
.apply(ProjectSubscribeServicePlugin.class)
@@ -50,6 +53,6 @@ public void apply(Project project) {
5053
t.setDescription("Validates that all TransportVersion references used in the project have an associated definition file");
5154
t.getReferencesFile().set(collectTask.get().getOutputFile());
5255
});
53-
project.getTasks().named(LifecycleBasePlugin.CHECK_TASK_NAME).configure(t -> t.dependsOn(validateTask));
56+
project.getTasks().named(PrecommitPlugin.PRECOMMIT_TASK_NAME).configure(t -> t.dependsOn(validateTask));
5457
}
5558
}

build-tools-internal/src/main/java/org/elasticsearch/gradle/internal/transport/TransportVersionResourcesPlugin.java

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,8 @@
1010
package org.elasticsearch.gradle.internal.transport;
1111

1212
import org.elasticsearch.gradle.internal.ProjectSubscribeServicePlugin;
13+
import org.elasticsearch.gradle.internal.conventions.precommit.PrecommitPlugin;
14+
import org.elasticsearch.gradle.internal.conventions.precommit.PrecommitTaskPlugin;
1315
import org.gradle.api.Plugin;
1416
import org.gradle.api.Project;
1517
import org.gradle.api.file.Directory;
@@ -26,6 +28,7 @@ public class TransportVersionResourcesPlugin implements Plugin<Project> {
2628
@Override
2729
public void apply(Project project) {
2830
project.getPluginManager().apply(LifecycleBasePlugin.class);
31+
project.getPluginManager().apply(PrecommitTaskPlugin.class);
2932
var psService = project.getPlugins().apply(ProjectSubscribeServicePlugin.class).getService();
3033
var resourceRoot = getResourceRoot(project);
3134

@@ -55,7 +58,7 @@ public void apply(Project project) {
5558
t.setDescription("Validates that all transport version resources are internally consistent with each other");
5659
t.getReferencesFiles().setFrom(tvReferencesConfig);
5760
});
58-
project.getTasks().named(LifecycleBasePlugin.CHECK_TASK_NAME).configure(t -> t.dependsOn(validateTask));
61+
project.getTasks().named(PrecommitPlugin.PRECOMMIT_TASK_NAME).configure(t -> t.dependsOn(validateTask));
5962

6063
var generateManifestTask = project.getTasks()
6164
.register("generateTransportVersionManifest", GenerateTransportVersionManifestTask.class, t -> {

0 commit comments

Comments
 (0)