Skip to content

Commit 6f37580

Browse files
bmartyclaude
andcommitted
Rename the Firebase consumer keep rules file
The -dontwarn is still required: firebase-messaging ships no consumer rules and we exclude firebase-analytics from its dependencies. Only rename the file to consumer-rules.keep and reword the comment, which embedded a local path. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
1 parent 2302795 commit 6f37580

3 files changed

Lines changed: 6 additions & 6 deletions

File tree

libraries/pushproviders/firebase/build.gradle.kts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ android {
2626

2727
buildTypes {
2828
getByName("release") {
29-
consumerProguardFiles("consumer-proguard-rules.pro")
29+
consumerProguardFiles("consumer-rules.keep")
3030
}
3131
register("nightly") {
32-
consumerProguardFiles("consumer-proguard-rules.pro")
32+
consumerProguardFiles("consumer-rules.keep")
3333
matchingFallbacks += listOf("release")
3434
}
3535
}

libraries/pushproviders/firebase/consumer-proguard-rules.pro

Lines changed: 0 additions & 4 deletions
This file was deleted.
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
# firebase-messaging references AnalyticsConnector, but we exclude firebase-analytics and
2+
# firebase-measurement-connector from its dependencies (see build.gradle.kts), so R8 reports it as a
3+
# missing class.
4+
-dontwarn com.google.firebase.analytics.connector.AnalyticsConnector

0 commit comments

Comments
 (0)