search

firebase / firebase-ios-sdk

Firebase SDK for Apple App Development
https://firebase.google.com
Apache License 2.0
5.52k stars 1.44k forks source link

[Firebase 12]: Audit deprecated public API for removal in upcoming major version release #12939

Open ncooke3 opened 3 months ago

ncooke3 commented 3 months ago

Description

Audit any deprecated API in the public API surface for removing in next major version release

Findable by running this command:

 git -p grep -e "__deprecated_msg" -e "__attribute__((deprecated))" -e "DEPRECATED_MSG_ATTRIBUTE" -e "deprecated:" -e "deprecated," -C 5 --color -- ':*.h' ':*.swift'  | cat

API Proposal

No response

Firebase Product(s)

AB Testing, Analytics, App Check, App Distribution, Authentication, Crashlytics, Database, DynamicLinks, Firestore, Functions, In-App Messaging, Installations, ML Model Downloader, Messaging, Performance, Remote Config, Storage, Vertex AI

ncooke3 commented 1 month ago

Deprecated Auth and Dynamic Links API is being left alone.

Deprecated Firestore CL: https://github.com/firebase/firebase-ios-sdk/blob/0acb7634c16695daff4e577bad876afc1379810d/Firestore/Source/Public/FirebaseFirestore/FIRFirestore.h#L132-L137

https://github.com/firebase/firebase-ios-sdk/blob/0acb7634c16695daff4e577bad876afc1379810d/Firestore/Source/Public/FirebaseFirestore/FIRFirestore.h#L161-L167

https://github.com/firebase/firebase-ios-sdk/blob/0acb7634c16695daff4e577bad876afc1379810d/Firestore/Source/Public/FirebaseFirestore/FIRFirestoreSettings.h#L56-L57

https://github.com/firebase/firebase-ios-sdk/blob/0acb7634c16695daff4e577bad876afc1379810d/Firestore/Source/Public/FirebaseFirestore/FIRFirestoreSettings.h#L69-L70

ncooke3 commented 1 month ago

Leaving this API alone for now. Will move this to Firebase 12 milestone.