-
-
Couldn't load subscription status.
- Fork 180
Added TvOS, MacOS, iOSX64 support #770
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
Conversation
Updated the build configuration to include support for tvOS by setting deployment targets and adjusting Kotlin opt-ins. Also, updated Gradle and library versions to their latest stable releases as of commit `9b4c5f6`. This change ensures compatibility with new platforms while leveraging the latest features and improvements in dependencies.
Updated the build configuration to include support for macOS by setting appropriate deployment targets. Enhanced platform coverage by adding macOS targets to existing iOS, tvOS, and Kotlin opt-ins. Updated Gradle and library versions to their latest stable releases, ensuring compatibility with new platforms while leveraging recent improvements in dependencies.
2cbd65d to
a98eaa9
Compare
joohnq
commented
Sep 13, 2025
Please come back with the iosX64
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR adds support for TvOS, MacOS, and iOSX64 platforms to the Firebase Kotlin SDK by extending the build configurations across all Firebase modules and updating the CI/CD pipeline to test these new platforms.
Key changes:
- Expanded Apple platform support from iOS-only to include TvOS, MacOS, and iOSX64
- Updated build configurations to support the new Apple platforms with appropriate deployment targets
- Added comprehensive test skipping configurations for the new platforms
Reviewed Changes
Copilot reviewed 24 out of 72 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| gradle.properties | Added test skip configuration properties for MacOS and TvOS platforms |
| gradle/libs.versions.toml | Added deployment target versions for TvOS (13.0) and MacOS (10.15) |
| Multiple build.gradle.kts files | Extended Kotlin multiplatform configurations to include new Apple targets and deployment settings |
| GitHub workflow files | Updated CI/CD pipelines to test MacOS and TvOS platforms with specific Xcode version |
| Test utility files | Minor import optimizations and platform detection logic updates |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
...supported targets for Firebase Crashlytics
@nbransby - want to review this?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 26 out of 74 changed files in this pull request and generated 3 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Copilot
AI
Oct 6, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The line break in the minSdk assignment makes the code less readable. Consider keeping the assignment on a single line or properly aligning the continuation.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 26 out of 74 changed files in this pull request and generated 2 comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 26 out of 74 changed files in this pull request and generated 1 comment.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
firebase-auth/build.gradle.kts
Outdated
Copilot
AI
Oct 7, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
[nitpick] The enableKeychainForTests() method calls are duplicated across multiple Apple platform targets. Consider extracting this into a helper function to reduce code duplication.
Uh oh!
There was an error while loading. Please reload this page.
This is taken from #729
Thank you for your contributions @sajidalidev
I have rebased and re-added iosX64.