Skip to content
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

build(deps): firebase emulator 9.16.4 fixes auth token issues #5595

Merged
merged 1 commit into from
Aug 13, 2021

Conversation

mikehardy
Copy link
Collaborator

Related firebase/firebase-tools#3674
Related firebase/firebase-tools#3671

Test Plan

ios and android e2e tests were both failing on this one, I got clean runs locally with the new release, if it passes in CI we're good


Think react-native-firebase is great? Please consider supporting the project with any of the below:

@vercel
Copy link

vercel bot commented Aug 13, 2021

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployments, click below or on the icon next to each commit.

react-native-firebase – ./

🔍 Inspect: https://vercel.com/invertase/react-native-firebase/85RT34HHCr5DrBS3rCszTaaXC7V1
✅ Preview: https://react-native-firebase-git-mikehardy-dependency-310d7b-invertase.vercel.app

react-native-firebase-next – ./website_modular

🔍 Inspect: https://vercel.com/invertase/react-native-firebase-next/BauuGu94RnA4wHMwZoVoEAiQW5zc
✅ Preview: Canceled

@mikehardy mikehardy added the Workflow: Pending Merge Waiting on CI or similar label Aug 13, 2021
@codecov
Copy link

codecov bot commented Aug 13, 2021

Codecov Report

Merging #5595 (18c66ec) into master (97f9090) will decrease coverage by 0.05%.
The diff coverage is n/a.

❗ Current head 18c66ec differs from pull request most recent head 6e0d191. Consider uploading reports for the commit 6e0d191 to get more accurate results

@@             Coverage Diff              @@
##             master    #5595      +/-   ##
============================================
- Coverage     67.34%   67.30%   -0.04%     
+ Complexity      989      987       -2     
============================================
  Files           202      202              
  Lines          9708     9708              
  Branches       1514     1514              
============================================
- Hits           6537     6533       -4     
- Misses         2754     2757       +3     
- Partials        417      418       +1     

@mikehardy mikehardy merged commit 53b846a into master Aug 13, 2021
@mikehardy mikehardy removed the Workflow: Pending Merge Waiting on CI or similar label Aug 13, 2021
@mikehardy mikehardy deleted the @mikehardy/dependency-updates branch August 17, 2021 22:35
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.

None yet

1 participant