fix(tests): bump FirebaseFirestore Podfile pin to 12.17.0 - #692
Open
demolaf wants to merge 4 commits into
Open
Conversation
The tests/ios and tests/macos Podfiles pinned FirebaseFirestore to the invertase/firestore-ios-sdk-frameworks fork tag 12.15.0, last bumped in #663. The resolved cloud_firestore plugin now requires Firebase/Firestore 12.17.0, which conflicts with the stale pin and fails pod install with a CocoaPods resolver error on every e2e run. CPRN-450
12.17.0 was already stale by the time CI ran: cloud_firestore's unpinned ^6.6.0 constraint resolved to 6.9.0 in this run, which requires Firebase/Firestore 12.18.0. CPRN-450
The tests app pinned org.jetbrains.kotlin.android to 2.1.0, but the transitively-resolved firebase-auth Android AAR (24.2.0, pulled in via the unpinned firebase_auth dependency) is compiled with Kotlin 2.3.0 metadata, which 2.1.0 cannot read: Module was compiled with an incompatible version of Kotlin. The binary version of its metadata is 2.3.0, expected version is 2.1.0. Same class of drift as the FirebaseFirestore Podfile pin, just on Android: an unpinned Firebase dependency moved ahead of a hardcoded version elsewhere in the test app. CPRN-450
Removes the caret ranges on firebase_auth, firebase_core, and cloud_firestore in tests/pubspec.yaml, replacing them with the exact versions currently resolved (firebase_auth 6.6.1, firebase_core 4.14.0, cloud_firestore 6.9.0). These three drive the native-side Firebase SDK version through firebase_core's FirebaseSDKVersion default (34.18.0 Android BOM, 12.18.0 iOS Firebase SDK), which is exactly what the tests/ios and tests/macos Podfile FirebaseFirestore tag and the Kotlin Gradle plugin version have to match. Left floating, any of the three can silently resolve to a newer version on any CI run and break e2e on a completely unrelated PR, which is what happened twice on this branch already. Everything else in the workspace stays unpinned, so the publishable firebase_ui_* packages keep testing against the latest compatible Firebase releases. CPRN-450
demolaf
added this pull request to stack #694
September 14, 2026 12:02
russellwheatley
approved these changes
Sep 14, 2026
russellwheatley
left a comment
Member
There was a problem hiding this comment.
Clean CI-unblock, pins match what's actually resolving. LGTM.
11 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The tests/ios and tests/macos Podfiles pin FirebaseFirestore to the invertase/firestore-ios-sdk-frameworks fork tag 12.15.0, last bumped in #663 back in July. Since then the resolved cloud_firestore plugin version now requires Firebase/Firestore 12.17.0, which conflicts with the stale pin and fails pod install with a CocoaPods resolver error on every e2e run, on main and on every open PR regardless of what it actually changes.
This bumps both Podfile pins to 12.17.0, which exists in the fork repo and matches what cloud_firestore currently resolves to.