Skip to content

Fix recently introduced build warning (#7745) #7755

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

Merged
merged 1 commit into from
Mar 19, 2021
Merged

Conversation

paulb777
Copy link
Member

This isn't worth a respin on its own, but if there is an RC2, please pull this in as well to fix a Firestore warning regression.

@google-oss-bot
Copy link

1 Warning
⚠️ Did you forget to add a changelog entry? (Add #no-changelog to the PR description to silence this warning.)

Generated by 🚫 Danger

@google-oss-bot
Copy link

Coverage Report

Affected SDKs

  • FirebaseAuth-iOS-FirebaseAuth.framework

    SDK overall coverage changed from 0.04% (a68e264) to 68.82% (afbe4f4) by +68.77%.

    Click to show coverage changes in 77 files.
    Filename Base (a68e264) Head (afbe4f4) Diff
    FIRActionCodeSettings.m 0.00% 100.00% +100.00%
    FIRAdditionalUserInfo.m 0.00% 92.50% +92.50%
    FIRAuth.m 0.21% 81.43% +81.21%
    FIRAuthAPNSToken.m 0.00% 100.00% +100.00%
    FIRAuthAPNSTokenManager.m 0.00% 50.52% +50.52%
    FIRAuthAppCredential.m 0.00% 91.30% +91.30%
    FIRAuthAppCredentialManager.m 0.00% 100.00% +100.00%
    FIRAuthBackend.m 0.00% 86.49% +86.49%
    FIRAuthCredential.m 0.00% 38.89% +38.89%
    FIRAuthDataResult.m 0.00% 41.38% +41.38%
    FIRAuthDispatcher.m 0.00% 100.00% +100.00%
    FIRAuthErrorUtils.m 0.00% 74.50% +74.50%
    FIRAuthGlobalWorkQueue.m 0.00% 100.00% +100.00%
    FIRAuthKeychainServices.m 0.00% 60.16% +60.16%
    FIRAuthNotificationManager.m 0.00% 84.38% +84.38%
    FIRAuthRequestConfiguration.m 0.00% 100.00% +100.00%
    FIRAuthSerialTaskQueue.m 0.00% 100.00% +100.00%
    FIRAuthSettings.m 0.00% 100.00% +100.00%
    FIRAuthStoredUserManager.m 0.00% 40.21% +40.21%
    FIRAuthTokenResult.m 0.00% 80.43% +80.43%
    FIRAuthURLPresenter.m 0.00% 48.39% +48.39%
    FIRAuthUserDefaults.m 0.00% 100.00% +100.00%
    FIRAuthWebUtils.m 0.00% 96.37% +96.37%
    FIRCreateAuthURIRequest.m 0.00% 61.29% +61.29%
    FIRCreateAuthURIResponse.m 0.00% 100.00% +100.00%
    FIRDeleteAccountRequest.m 0.00% 100.00% +100.00%
    FIRDeleteAccountResponse.m 0.00% 100.00% +100.00%
    FIREmailAuthProvider.m 0.00% 54.55% +54.55%
    FIREmailLinkSignInRequest.m 0.00% 91.30% +91.30%
    FIREmailLinkSignInResponse.m 0.00% 100.00% +100.00%
    FIREmailPasswordAuthCredential.m 0.00% 34.04% +34.04%
    FIRFacebookAuthCredential.m 0.00% 38.46% +38.46%
    FIRFacebookAuthProvider.m 0.00% 37.50% +37.50%
    FIRGetAccountInfoRequest.m 0.00% 100.00% +100.00%
    FIRGetAccountInfoResponse.m 0.00% 78.38% +78.38%
    FIRGetOOBConfirmationCodeRequest.m 0.00% 87.40% +87.40%
    FIRGetOOBConfirmationCodeResponse.m 0.00% 100.00% +100.00%
    FIRGetProjectConfigRequest.m 0.00% 100.00% +100.00%
    FIRGetProjectConfigResponse.m 0.00% 64.29% +64.29%
    FIRGitHubAuthCredential.m 0.00% 38.46% +38.46%
    FIRGitHubAuthProvider.m 0.00% 37.50% +37.50%
    FIRGoogleAuthCredential.m 0.00% 40.00% +40.00%
    FIRGoogleAuthProvider.m 0.00% 37.50% +37.50%
    FIRIdentityToolkitRequest.m 0.00% 92.21% +92.21%
    FIRMultiFactor.m 0.00% 10.79% +10.79%
    FIROAuthCredential.m 0.00% 51.35% +51.35%
    FIROAuthProvider.m 0.00% 85.11% +85.11%
    FIRPhoneAuthCredential.m 0.00% 31.37% +31.37%
    FIRPhoneAuthProvider.m 0.00% 55.46% +55.46%
    FIRResetPasswordRequest.m 0.00% 89.47% +89.47%
    FIRResetPasswordResponse.m 0.00% 100.00% +100.00%
    FIRSecureTokenRequest.m 0.00% 53.33% +53.33%
    FIRSecureTokenService.m 0.00% 84.11% +84.11%
    FIRSendVerificationCodeRequest.m 0.00% 93.10% +93.10%
    FIRSendVerificationCodeResponse.m 0.00% 57.14% +57.14%
    FIRSetAccountInfoRequest.m 0.00% 96.61% +96.61%
    FIRSetAccountInfoResponse.m 0.00% 100.00% +100.00%
    FIRSignInWithGameCenterRequest.m 0.00% 100.00% +100.00%
    FIRSignInWithGameCenterResponse.m 0.00% 100.00% +100.00%
    FIRSignUpNewUserRequest.m 0.00% 94.44% +94.44%
    FIRSignUpNewUserResponse.m 0.00% 100.00% +100.00%
    FIRTwitterAuthCredential.m 0.00% 40.00% +40.00%
    FIRTwitterAuthProvider.m 0.00% 37.50% +37.50%
    FIRUser.m 0.00% 69.33% +69.33%
    FIRUserInfoImpl.m 0.00% 93.88% +93.88%
    FIRUserMetadata.m 0.00% 86.36% +86.36%
    FIRVerifyAssertionRequest.m 0.00% 90.79% +90.79%
    FIRVerifyAssertionResponse.m 0.00% 87.84% +87.84%
    FIRVerifyClientRequest.m 0.00% 100.00% +100.00%
    FIRVerifyClientResponse.m 0.00% 100.00% +100.00%
    FIRVerifyCustomTokenRequest.m 0.00% 89.47% +89.47%
    FIRVerifyCustomTokenResponse.m 0.00% 100.00% +100.00%
    FIRVerifyPasswordRequest.m 0.00% 94.12% +94.12%
    FIRVerifyPasswordResponse.m 0.00% 65.38% +65.38%
    FIRVerifyPhoneNumberRequest.m 0.00% 82.76% +82.76%
    FIRVerifyPhoneNumberResponse.m 0.00% 81.25% +81.25%
    NSData+FIRBase64.m 0.00% 100.00% +100.00%
  • FirebaseDatabase-iOS-FirebaseDatabase.framework

    SDK overall coverage changed from 56.93% (a68e264) to 56.78% (afbe4f4) by -0.15%.

    Filename Base (a68e264) Head (afbe4f4) Diff
    FSRWebSocket.m 41.60% 39.77% -1.83%
  • FirebaseDynamicLinks-iOS-FirebaseDynamicLinks.framework

    SDK overall coverage changed from 76.69% (a68e264) to 75.64% (afbe4f4) by -1.05%.

    Filename Base (a68e264) Head (afbe4f4) Diff
    FDLUtilities.m 97.35% 90.15% -7.20%
    FIRDLDefaultRetrievalProcessV2.m 73.96% 72.92% -1.04%
    FIRDynamicLink.m 81.90% 80.17% -1.72%
    FIRDynamicLinkNetworking.m 81.67% 80.83% -0.83%
  • FirebaseFirestore-iOS-FirebaseFirestore.framework

    SDK overall coverage changed from 88.56% (a68e264) to 88.55% (afbe4f4) by -0.02%.

    Filename Base (a68e264) Head (afbe4f4) Diff
    ordered_code.cc 93.24% 92.68% -0.56%
    task.cc 96.41% 95.21% -1.20%

Test Logs

@paulb777 paulb777 merged commit 76ce160 into release-7.9 Mar 19, 2021
@paulb777 paulb777 deleted the pb-cp-7745 branch March 19, 2021 17:00
maksymmalyhin added a commit that referenced this pull request Mar 23, 2021
* Update versions for Release 7.9.0

* Fix pod lib lint for watchOS + add CI test (#7740)

* Fix pod lib lint for watchOS

* Add watchOS build test.

* Code review feedback

* Typo and formatting

* Typo and formatting

* fix typo

* Add missing test util files to tests targets

Co-authored-by: Maksym Malyhin <[email protected]>

* Resolve new SPM package install warning (#7741)

* Cherry pick: Analytics 7.9.0 RC1 fix (#7750) (#7751)

* Analytics 7.9.0 RC1 fix (#7750)

* Analytics 7.9.0 RC1 (#7749)

Co-authored-by: Allen Vicencio <[email protected]>

* Fix recently introduced build warning (#7745) (#7755)

* Analytics 7.9.0 RC2 (#7757) (#7759)

Co-authored-by: Allen Vicencio <[email protected]>

* Revert "Add a warning to upload-symbols when it detects an invalid dSYM (#7705)" (#7761) (#7762)

This reverts commit a3eb3b5.

* Update SPM analytics for 7.9.0 (#7774)

* M92: update FirebaseAnalytics binary link for SPM

* SPM: Update GoogleAppMeasurement version

Co-authored-by: Ryan Wilson <[email protected]>
Co-authored-by: Paul Beusterien <[email protected]>
Co-authored-by: Allen Vicencio <[email protected]>
Co-authored-by: Sam Edson <[email protected]>
@firebase firebase locked and limited conversation to collaborators Apr 19, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants