ci: bring :app and :e2eTest under the Android Lint gate

.github/workflows/lint.yml

@@ -31,8 +31,8 @@ jobs:
           java-version: '21'
           distribution: 'temurin'
 
-      # lintAll gates :proguard-tests, which applies the google-services plugin and
-      # will not configure without this file. Mirrors what scripts/build.sh copies.
+      # lintAll gates :app and :proguard-tests, both of which apply the google-services
+      # plugin and will not configure without this file. Mirrors what build.sh copies.
       - name: Copy google-services.json
         run: |
           cp library/google-services.json app/google-services.json

app/build.gradle.kts

@@ -39,6 +39,26 @@ android {
             }
         }
     }
+
+    lint {
+        // Module specific
+        disable += mutableSetOf(
+            // Reads the root wrapper, but only the application module analyses it, so it
+            // belongs here rather than in the shared policy. For reproducible builds.
+            "AndroidGradlePluginVersion",
+            // The demos log their auth callbacks unconditionally on purpose — watching
+            // logcat is how you see one fire. Unlike :auth's, none of these log user data.
+            "LogConditional",
+            // Glide's KSP processor does not generate GlideApp, which the storage demo and
+            // storage/README.md are both written around. Migration tracked separately.
+            "KaptUsageInsteadOfKsp",
+            // A themed icon needs a flat silhouette drawn for the purpose. The only
+            // candidate here is ic_launcher_foreground, whose opaque region is a solid
+            // plate, so it tints to a featureless block — worse than no monochrome layer.
+            "MonochromeLauncherIcon"
+        )
+    }
+
     compileOptions {
         sourceCompatibility = JavaVersion.VERSION_17
         targetCompatibility = JavaVersion.VERSION_17

app/src/main/AndroidManifest.xml

@@ -14,15 +14,20 @@
         android:usesCleartextTraffic="true">
         <activity
             android:name="com.firebaseui.android.demo.MainActivity"
-            android:label="@string/app_name"
             android:exported="true"
             android:theme="@style/Theme.FirebaseUIAndroid">
             <intent-filter>
                 <action android:name="android.intent.action.MAIN" />
                 <category android:name="android.intent.category.LAUNCHER" />
             </intent-filter>
 
-            <intent-filter android:autoVerify="true">
+            <!--
+                The host is a placeholder the demo's own Firebase project fills in, so lint
+                can neither read it as a domain nor resolve it over the network.
+            -->
+            <intent-filter
+                android:autoVerify="true"
+                tools:ignore="AppLinkUrlError,AppLinksAutoVerify">
                 <action android:name="android.intent.action.VIEW" />
                 <category android:name="android.intent.category.DEFAULT" />
                 <category android:name="android.intent.category.BROWSABLE" />

app/src/main/java/com/firebaseui/android/demo/database/DatabaseDemoActivity.kt

@@ -33,6 +33,7 @@ import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import com.firebase.ui.database.paging.DatabasePagingOptions
 import com.firebase.ui.database.paging.FirebaseRecyclerPagingAdapter
+import com.firebaseui.android.demo.R
 import com.google.firebase.auth.FirebaseAuth
 import com.google.firebase.database.DatabaseReference
 import com.google.firebase.database.FirebaseDatabase
@@ -122,7 +123,8 @@ class ScoreAdapter(options: DatabasePagingOptions<ScoreItem>) :
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int) = ScoreViewHolder(parent)
 
     override fun onBindViewHolder(holder: ScoreViewHolder, position: Int, model: ScoreItem) {
-        (holder.itemView as TextView).text = "${model.name}  —  score: ${model.score}"
+        val row = holder.itemView as TextView
+        row.text = row.context.getString(R.string.demo_score_row, model.name, model.score)
     }
 }
 

app/src/main/java/com/firebaseui/android/demo/firestore/FirestoreDemoActivity.kt

@@ -33,6 +33,7 @@ import androidx.recyclerview.widget.LinearLayoutManager
 import androidx.recyclerview.widget.RecyclerView
 import com.firebase.ui.firestore.paging.FirestorePagingAdapter
 import com.firebase.ui.firestore.paging.FirestorePagingOptions
+import com.firebaseui.android.demo.R
 import com.google.firebase.auth.FirebaseAuth
 import com.google.firebase.firestore.CollectionReference
 import com.google.firebase.firestore.FirebaseFirestore
@@ -129,7 +130,8 @@ class ScoreAdapter(options: FirestorePagingOptions<ScoreItem>) :
     override fun onCreateViewHolder(parent: ViewGroup, viewType: Int) = ScoreViewHolder(parent)
 
     override fun onBindViewHolder(holder: ScoreViewHolder, position: Int, model: ScoreItem) {
-        (holder.itemView as TextView).text = "${model.name}  —  score: ${model.score}"
+        val row = holder.itemView as TextView
+        row.text = row.context.getString(R.string.demo_score_row, model.name, model.score)
     }
 }
 

app/src/main/res/values/strings.xml

@@ -1,10 +1,19 @@
-<resources>
+<resources xmlns:tools="http://schemas.android.com/tools">
     <string name="app_name">FirebaseUI Demo</string>
 
     <string name="firebase_web_host" translatable="false">CHANGE-HERE</string>
 
-    <!-- Facebook SDK Configuration -->
-    <string name="facebook_application_id" translatable="false">APP-ID</string>
-    <string name="facebook_login_protocol_scheme" translatable="false">fbAPP-ID</string>
-    <string name="facebook_client_token" translatable="false">CHANGE-HERE</string>
-</resources>
+    <!-- One row of the paging demos: %1$s is the player name, %2$d their score. -->
+    <string name="demo_score_row">%1$s  —  score: %2$d</string>
+
+    <!--
+        The Facebook SDK reads these through the auth module's manifest and provider
+        validation, so lint cannot see them being used from this module.
+    -->
+    <string name="facebook_application_id" translatable="false"
+        tools:ignore="UnusedResources">APP-ID</string>
+    <string name="facebook_login_protocol_scheme" translatable="false"
+        tools:ignore="UnusedResources">fbAPP-ID</string>
+    <string name="facebook_client_token" translatable="false"
+        tools:ignore="UnusedResources">CHANGE-HERE</string>
+</resources>

auth/build.gradle.kts

@@ -37,14 +37,6 @@ android {
     }
 
     lint {
-        // Common lint options across all modules
-        disable += mutableSetOf(
-            "IconExpectedSize",
-            "InvalidPackage", // Firestore uses GRPC which makes lint mad
-            "NewerVersionAvailable", "GradleDependency", // For reproducible builds
-            "SelectableText", "SyntheticAccessor" // We almost never care about this
-        )
-
         // Module specific
         disable += mutableSetOf(
             "UnusedQuantity",
@@ -64,10 +56,6 @@ android {
             "LogConditional"
         )
 
-        checkAllWarnings = true
-        warningsAsErrors = true
-        abortOnError = true
-
         // Pre-existing debt only: 168 localization findings (CPRN-432) and 12 Compose
         // correctness findings (CPRN-436). Every entry is suppressed; new ones still fail.
         baseline = file("lint-baseline.xml")