Tweak formatting

elastic · May 27, 2020 · c4c2548 · c4c2548
1 parent d42cb77
commit c4c2548
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 3 deletions.
diff --git a/buildSrc/src/main/java/org/elasticsearch/gradle/ElasticsearchJavaPlugin.java b/buildSrc/src/main/java/org/elasticsearch/gradle/ElasticsearchJavaPlugin.java
@@ -487,6 +487,8 @@ static void configureJars(Project project) {
                     // we put all our distributable files under distributions
                     jarTask.getDestinationDirectory().set(new File(project.getBuildDir(), "distributions"));
                     // fixup the jar manifest
+                    // Explicitly using an Action interface as java lambdas
+                    // are not supported by Gradle up-to-date checks
                     jarTask.doFirst(new Action<Task>() {
                         @Override
                         public void execute(Task task) {

diff --git a/buildSrc/src/main/java/org/elasticsearch/gradle/precommit/DependencyLicensesTask.java b/buildSrc/src/main/java/org/elasticsearch/gradle/precommit/DependencyLicensesTask.java
@@ -227,9 +227,9 @@ public void checkDependencies() throws IOException, NoSuchAlgorithmException {
 
     }
 
-    // this is just a marker output folder to allow this task being up-to-date
-    // the check logic is exception driven so a failed task will not be defined
-    // by this output but when successful we can safely mark the task as up-to-date
+    // This is just a marker output folder to allow this task being up-to-date.
+    // The check logic is exception driven so a failed tasks will not be defined
+    // by this output but when successful we can safely mark the task as up-to-date.
     @OutputDirectory
     public File getOutputMarker() {
         return new File(getProject().getBuildDir(), "dependencyLicense");