Move precommit dependencies to precommit for clarity.

gradle/validation/forbidden-apis.gradle

@@ -64,9 +64,8 @@ allprojects { prj ->
       ]
     }
 
-    // Attach validation to check and precommit tasks.
+    // Attach validation to check task.
     check.dependsOn forbiddenApisMain, forbiddenApisTest
-    precommit.dependsOn forbiddenApisMain, forbiddenApisTest
 
     // Disable sysout signatures for these projects.
     if (prj.path in [

gradle/validation/git-status.gradle

@@ -49,7 +49,4 @@ configure(rootProject) {
       }
     }
   }
-
-  // Attach to precommit.
-  precommit.dependsOn checkWorkingCopyPristine
 }

gradle/validation/jar-checks.gradle

@@ -53,7 +53,6 @@ allprojects {
     description = "Apply all dependency/ license checks."
   }
   check.dependsOn(licenses)
-  precommit.dependsOn(licenses)
 }
 
 subprojects {

gradle/validation/precommit.gradle

@@ -1,12 +1,23 @@
 // This adds top-level 'precommit' task.
-// Validation tasks attach themselves to it in their own
-// respective build scripts.
 
 configure(rootProject) {
   task precommit() {
     group = 'Precommit'
     description = "All precommit checks"
 
+    // Root-level dependencies.
     dependsOn verifyLocks
+    dependsOn ":versionsPropsAreSorted"
+    dependsOn ":checkWorkingCopyPristine"
+
+    // Attach tasks from other projects using lazy collection
+    // as they may not yet be defined.
+    dependsOn allprojects.collect { prj ->
+      prj.tasks.matching { task -> task.name in [
+          "forbiddenApisMain",
+          "forbiddenApisTest",
+          "licenses",
+      ]}
+    }
   }
 }

gradle/validation/versions-props-sorted.gradle

@@ -14,6 +14,4 @@ configure(rootProject) {
       }
     }
   }
-
-  precommit.dependsOn versionsPropsAreSorted
 }