Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
|
Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 15 days until this gets closed automatically |
|
Hi there! 👋 sorry for the delay reviewing this, could you help me understand why this is needed? You included no description at all that would explain the motivation for the change, and why you used the approach you did Additionally, it doesn't seem to work - the CI builds failed I'm going to put this in draft pending more details and a clean CI run Thanks |
mikehardy
left a comment
There was a problem hiding this comment.
Needs description + green CI
|
Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 15 days until this gets closed automatically |
Description
Related issues
Release Summary
Checklist
AndroidiOSOther(macOS, web)e2etests added or updated inpackages/\*\*/e2ejesttests added or updated inpackages/\*\*/__tests__Test Plan
Think
react-native-firebaseis great? Please consider supporting the project with any of the below:React Native FirebaseandInvertaseon Twitter