Skip to content

feat: enable Firebase 13 compatibility, remove unused dependency, and address build warnings - #1406

Open
ncooke3 wants to merge 6 commits into
mainfrom
nc.f13
Open

ncooke3 wants to merge 6 commits into
mainfrom
nc.f13

Conversation

@ncooke3

@ncooke3 ncooke3 commented Oct 9, 2026 •

Copy link
Copy Markdown
Member

After merging, this would need a SPM-only v16.2.0 tag and release page creation.

🎯 Usefulness is blocked until GSI 10.1.0 that will include Firebase 13 compatibility: google/GoogleSignIn-iOS#626

CocoaPods should no longer be published in alignment with firebase-ios-sdk.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates dependency version ranges in Package.swift and Package.resolved, adds Sendable conformances to authentication context types, removes redundant await keywords and argument labels, and adds .macOS(.v12) to the supported platforms in Package.swift. Feedback on the changes highlights that declaring macOS support in Package.swift will cause compilation failures because several targets use iOS-only APIs without conditional compilation guards.

Comment thread Package.swift

@paulb777 paulb777 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Follow-Up Note (Post-Firebase 13 / GSI 10.1.0 Release)

  • Once firebase-ios-sdk 13.0.0 and GoogleSignIn-iOS 10.1.0 are tagged, consider bumping at least one sample/test Xcode project (samples/swift/FirebaseUI-demo-swift.xcodeproj is currently upToNextMajorVersion from 12.14.0, and e2eTest/FirebaseSwiftUIExample.xcodeproj is upToNextMajorVersion from 11.12.0) and/or Package.resolved so CI builds and tests against Firebase 13 directly.

Comment thread Package.swift Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants