Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Keep FirebaseCore, FirebaseCoreExtension and FirebaseCoreInternal pod versions aligned to latest #13953

Merged
merged 2 commits into from
Oct 23, 2024

Conversation

paulb777
Copy link
Member

Prevent future breakages like #13942

There is no real value in letting these versions diverge and this better matches SPM

@google-oss-bot
Copy link

1 Warning
⚠️ Did you forget to add a changelog entry? (Add #no-changelog to the PR description to silence this warning.)

Generated by 🚫 Danger

@paulb777
Copy link
Member Author

CI failures look to be related to SpecStaging revert for 11.4.x testing

@ncooke3
Copy link
Member

ncooke3 commented Oct 23, 2024

I think we should also change FirebaseCoreExtension.podspec to have a strict dependency on FirebaseCore as well:

s.dependency 'FirebaseCore', '~> 11.0'

Because like in the Flutter bug, FirebaseAuth v11.2's floating depending on FirebaseCoreExtension allowed it depend on a newer FirebaseCoreExtension. If FirebaseCoreExtension strictly depended on FirebaseCore, it would prevent this float when using pods from the Firebase.podspec. because the Firebase.podspec strictly depends on a FirebaseCore version. Flutter currently uses the Firebase pod: https://github.com/firebase/flutterfire/blob/b5511588fed0762c102cf1e0b96a3f2e365ae2dd/packages/firebase_auth/firebase_auth/ios/firebase_auth.podspec#L34-L35

@paulb777 paulb777 merged commit 36da449 into main Oct 23, 2024
392 of 400 checks passed
@paulb777 paulb777 deleted the pb-lockstep-coreinternal branch October 23, 2024 17:28
@ncooke3
Copy link
Member

ncooke3 commented Oct 23, 2024

Thanks! Will re-stage and re-tag.

@firebase firebase locked and limited conversation to collaborators Nov 23, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.