Skip to content

infra: Remove ObjC quickstarts #1806

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 160 commits into from
Nov 14, 2025
Merged

infra: Remove ObjC quickstarts #1806

merged 160 commits into from
Nov 14, 2025

Conversation

@ncooke3
Copy link
Member

@ncooke3 ncooke3 commented Nov 12, 2025 •
edited
Loading

Corresponding Firebase PR: firebase/firebase-ios-sdk#15478

This PR migrate the quickstart examples from CocoaPods to Swift Package Manager (SPM). This involves several related changes:

  • SPM Migration: Most of the quickstart projects have been updated to use SPM for managing Firebase dependencies, moving away from CocoaPods.
  • Project and Scheme Renaming: Many Xcode projects, schemes, and targets have been renamed to be more consistent and simplify testing infrastructure. For example, suffixes like "Swift" have been removed from target
    names (e.g., InAppMessagingExampleSwift to InAppMessagingExample).
  • CI Updates: The GitHub Actions workflows have been significantly updated to build and test the quickstarts using SPM.
  • Legacy Code Removal: The Objective-C quickstart examples have been removed to focus on the Swift versions.
  • Bug Fixes and Refinements: Various smaller fixes and improvements have been made, such as fixing test target configurations, addressing warnings, and removing unused dependencies like Reachability.

morganchen12 and others added 30 commits January 16, 2025 14:24
@morganchen12
move Analytics off of CocoaPods
4827cef
@morganchen12
move app distribution to spm
e7319f4
@morganchen12
move Auth to swift 6 and spm
80e47a6
@morganchen12
move remoteconfig to spm
dfffbee
@morganchen12
update Firebase package
53a5961
@morganchen12
move database to spm
ec46631
@morganchen12
migrate Firestore sample to spm
73678e3
@morganchen12
update package version
a0dd61d
@morganchen12
migrate to spm and remove fdl
af5b0c8
@morganchen12
migrate installations to spm and remove todos
2b72b65
@morganchen12
migrate messaging to spm
6d27fcb
@morganchen12
move perf to spm
30a3934
@morganchen12
update firebase version
a7b7e8e
@morganchen12
use spm for tests
1df0e6e
@morganchen12
move rc test to spm
ff50fbd
@morganchen12
fix database tests
1f902bb
@morganchen12
fix firestore build
4991feb
@morganchen12
fix iam and installations tests
79b0b98
@morganchen12
fix style and remaining tests
e4d4ea9
@ncooke3
Merge branch 'main' into nc/spm
ad94bb3
@ncooke3
Merge pull request #1755 from firebase/nc/spm
5c1c372 
Merge main
@ncooke3
Update authentication.yml
b132aec
@ncooke3
refactor(storage): Move storage qs to root of storage directory
b650880
@ncooke3
Merge pull request #1756 from firebase/nc/spm-1
1f6a43a 
refactor(storage): Move storage qs to root of storage directory
@ncooke3
scheme logic
0b31b4a
@ncooke3
Merge pull request #1758 from firebase/nc/spm-2
a803127 
fix(infra): Update scripts/test.sh to dynamically get scheme
@ncooke3
abtesting warning fixes
7f273e3
@ncooke3
maybe revert: source_root
7f27d68
@ncooke3
Revert "maybe revert: source_root"
f4731f4 
This reverts commit 7f27d68.
@ncooke3
fix storage plist location
1564f34
ncooke3
ncooke3 commented Nov 12, 2025
View reviewed changes
Copy link
Member Author

@ncooke3 ncooke3 left a comment

Choose a reason for hiding this comment

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

Self review

@ncooke3 ncooke3 mentioned this pull request Nov 12, 2025
Merged
5 tasks
@ncooke3 ncooke3 requested a review from paulb777 November 14, 2025 18:56
@ncooke3 ncooke3 merged commit c39fe8d into main Nov 14, 2025
37 of 38 checks passed
@ncooke3 ncooke3 deleted the nc/quickstarts branch November 14, 2025 21:34
paulb777 added a commit that referenced this pull request Nov 15, 2025
@paulb777
Revert "Merge pull request #1806 from firebase/nc/quickstarts"
2941866 
This reverts commit c39fe8d, reversing
changes made to 251909a.
@gemini-code-assist gemini-code-assist bot mentioned this pull request Nov 15, 2025
Merged
paulb777 added a commit that referenced this pull request Nov 16, 2025
@paulb777
Revert "Merge pull request #1806 from firebase/nc/quickstarts" (#1811)
0b2a3e7 
This reverts commit c39fe8d, reversing
changes made to 251909a.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@morganchen12 morganchen12 morganchen12 approved these changes

@andrewheard andrewheard Awaiting requested review from andrewheard

@daymxn daymxn Awaiting requested review from daymxn

@paulb777 paulb777 Awaiting requested review from paulb777

+1 more reviewer

@gemini-code-assist gemini-code-assist[bot] gemini-code-assist[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@ncooke3 @morganchen12