build(ci): auto-opt in new modules to CI lint

By defining a 'lintAll' gradle task

Related to issue 21063

Assisted-by: Claude Opus 4.7 - all but comments

Author: david-allison

.github/workflows/README.md

@@ -11,13 +11,13 @@ following scripts before submitting a PR.
 
 Alternately, you may run the actions on your fork of `Anki-Android`.
 
-| **Job**                                                                                                    | **Command**                                                                                                           | **Comments**                                              |
-|------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------------------------------------------------------------------|-----------------------------------------------------------|
-| [Lint (Kotlin)](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/lint.yml)            | `./gradlew lintPlayDebug :api:lintDebug :libanki:lintDebug ktLintCheck lintVitalFullRelease lint-rules:test --daemon` | Android lint rules, formatting and tests for lint rules   |
-| [Lint (JavaScript)](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/lint.yml)        | See script                                                                                                            | Prettier, lint & code formatting                          |
-| [Unit Tests](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/tests_unit.yml)         | `./gradlew jacocoUnitTestReport --daemon`                                                                             | Unit tests for the Android Project                        |
-| [Emulator Tests](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/tests_emulator.yml) | `TEST_RELEASE_BUILD=true ./gradlew jacocoAndroidTestReport --daemon`                                                  | Emulator tests for the Android Project                    |
-| [CodeQL](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/codeql.yml)                 | N/A                                                                                                                   | GitHub-only check.<br/>[Docs](https://codeql.github.com/) |
+| **Job**                                                                                                    | **Command**                                                                   | **Comments**                                              |
+|------------------------------------------------------------------------------------------------------------|-------------------------------------------------------------------------------|-----------------------------------------------------------|
+| [Lint (Kotlin)](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/lint.yml)            | `./gradlew lintAll ktLintCheck lintVitalFullRelease lint-rules:test --daemon` | Android lint rules, formatting and tests for lint rules   |
+| [Lint (JavaScript)](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/lint.yml)        | See script                                                                    | Prettier, lint & code formatting                          |
+| [Unit Tests](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/tests_unit.yml)         | `./gradlew jacocoUnitTestReport --daemon`                                     | Unit tests for the Android Project                        |
+| [Emulator Tests](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/tests_emulator.yml) | `TEST_RELEASE_BUILD=true ./gradlew jacocoAndroidTestReport --daemon`          | Emulator tests for the Android Project                    |
+| [CodeQL](https://github.com/ankidroid/Anki-Android/blob/main/.github/workflows/codeql.yml)                 | N/A                                                                           | GitHub-only check.<br/>[Docs](https://codeql.github.com/) |
 
 ## Other Workflows
 

.github/workflows/lint.yml

@@ -53,13 +53,10 @@ jobs:
           command: ./gradlew :AnkiDroid:compilePlayDebugJavaWithJavac compileLint lint-rules:compileTestJava --daemon
 
       - name: Run Lint Debug
-        # "lint" is run under the 'Amazon' flavor, so slow in CI
-        # "lintPlayRelease" doesn't test androidTest
-        # "lintPlayDebug" doesn't test the API
+        # "lintAll" opts in all but :AnkiDroid to lint, pinning :AnkiDroid to lintPlayDebug
         # "lintVitalFullRelease": if `main` resources are only used in `androidTest` (#15741)
-
         # update workflows/README.md when modifying this line
-        run: ./gradlew lintPlayDebug :api:lintDebug :libanki:lintDebug ktLintCheck lintVitalFullRelease lint-rules:test --daemon
+        run: ./gradlew lintAll ktLintCheck lintVitalFullRelease lint-rules:test --daemon
 
   # update workflows/README.md when modifying commands
   lintJavascript:

build.gradle.kts

@@ -135,6 +135,22 @@ subprojects {
     }
 }
 
+// Opt all modules in to lint (with :AnkiDroid pinned to one flavor: 'Play')
+val lintAll = tasks.register("lintAll") {
+    group = "verification"
+    description = "Runs lint on every module."
+    dependsOn(":AnkiDroid:lintPlayDebug") // specify 'Play' explicitly so other flavors don't run
+}
+
+subprojects {
+    if (path == ":AnkiDroid") return@subprojects // pinned above to avoid other flavors
+    afterEvaluate {
+        // 'lintDebug' applies to all Android modules; 'lint' for all JVM modules
+        (tasks.findByName("lintDebug") ?: tasks.findByName("lint"))
+            ?.let { lintAll.configure { dependsOn(it) } }
+    }
+}
+
 val jvmVersion = Jvm.current().javaVersion?.majorVersion.parseIntOrDefault(defaultValue = 0)
 val minSdk: String = libs.versions.minSdk.get()
 val jvmVersionLowerBound = 21