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

m139 release #5466

Open
wants to merge 13 commits into
base: releases/m139
Choose a base branch
from
Open

m139 release #5466

wants to merge 13 commits into from

Conversation

github-actions[bot]
Copy link
Contributor

Auto-generated PR for release m139

@google-cla
Copy link

google-cla bot commented Oct 20, 2023

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@github-actions
Copy link
Contributor Author

github-actions bot commented Oct 20, 2023

Unit Test Results

     936 files  +     936       936 suites  +936   42m 54s ⏱️ + 42m 54s
  5 097 tests +  5 097    5 076 ✔️ +  5 076  21 💤 +21  0 ±0 
10 279 runs  +10 279  10 237 ✔️ +10 237  42 💤 +42  0 ±0 

Results for commit 1c9f06f. ± Comparison against base commit 250c69f.

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Oct 20, 2023

Size Report 1

Affected Products

  • firebase-common

    TypeBase (250c69f)Merge (109a2b5)Diff
    aar96.0 kB107 kB+11.2 kB (+11.7%)
    apk (release)1.64 MB1.65 MB+4.79 kB (+0.3%)
  • firebase-common-ktx

    TypeBase (250c69f)Merge (109a2b5)Diff
    apk (release)1.64 MB1.65 MB+5.18 kB (+0.3%)
  • firebase-crashlytics

    TypeBase (250c69f)Merge (109a2b5)Diff
    aar367 kB377 kB+10.1 kB (+2.8%)
    apk (aggressive)435 kB437 kB+1.74 kB (+0.4%)
    apk (release)2.18 MB2.18 MB+2.39 kB (+0.1%)
  • firebase-crashlytics-ktx

    TypeBase (250c69f)Merge (109a2b5)Diff
    apk (aggressive)435 kB437 kB+1.69 kB (+0.4%)
    apk (release)2.18 MB2.18 MB+2.82 kB (+0.1%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/6wpgbzcYkR.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Oct 20, 2023

Coverage Report 1

Affected Products

  • firebase-appdistribution

    Overall coverage changed from ? (250c69f) to 75.99% (109a2b5) by ?.

    68 individual files with coverage change

    FilenameBase (250c69f)Merge (109a2b5)Diff
    AabUpdater.java?98.36%?
    AabUpdater_Factory.java?0.00%?
    ApkInstaller.java?100.00%?
    ApkInstaller_Factory.java?0.00%?
    ApkUpdater.java?92.63%?
    ApkUpdater_Factory.java?0.00%?
    AppDistributionReleaseImpl.java?100.00%?
    AppDistributionReleaseInternal.java?100.00%?
    AppDistroComponent.java?0.00%?
    AppDistroComponent_MainModule_BindContentResolverFactory.java?0.00%?
    AppIconSource.java?84.62%?
    AppIconSource_Factory.java?100.00%?
    AutoValue_AppDistributionReleaseImpl.java?65.45%?
    AutoValue_AppDistributionReleaseInternal.java?71.58%?
    AutoValue_ImageUtils_ImageSize.java?35.00%?
    AutoValue_TesterApiDisabledErrorDetails.java?29.41%?
    AutoValue_TesterApiDisabledErrorDetails_HelpLink.java?54.17%?
    AutoValue_UpdateProgressImpl.java?65.96%?
    DaggerAppDistroComponent.java?80.56%?
    DevModeDetector.java?9.09%?
    DevModeDetector_Factory.java?100.00%?
    ErrorMessages.java?0.00%?
    FeedbackActivity.java?3.39%?
    FeedbackActivity_MembersInjector.java?0.00%?
    FeedbackSender.java?84.48%?
    FeedbackSender_Factory.java?0.00%?
    FeedbackTrigger.java?61.54%?
    FirebaseAppDistributionExceptions.java?80.00%?
    FirebaseAppDistributionFileProvider.java?0.00%?
    FirebaseAppDistributionImpl.java?89.89%?
    FirebaseAppDistributionImpl_Factory.java?0.00%?
    FirebaseAppDistributionLifecycleNotifier.java?91.49%?
    FirebaseAppDistributionLifecycleNotifier_Factory.java?0.00%?
    FirebaseAppDistributionNotificationsManager.java?88.89%?
    FirebaseAppDistributionNotificationsManager_Factory.java?0.00%?
    FirebaseAppDistributionRegistrar.java?95.83%?
    FirebaseAppDistributionTesterApiClient.java?88.78%?
    FirebaseAppDistributionTesterApiClient_Factory.java?0.00%?
    HttpsUrlConnectionFactory.java?50.00%?
    HttpsUrlConnectionFactory_Factory.java?100.00%?
    ImageUtils.java?100.00%?
    InstallActivity.java?2.67%?
    LogWrapper.java?86.67%?
    NewReleaseFetcher.java?86.67%?
    NewReleaseFetcher_Factory.java?0.00%?
    PackageInfoUtils.java?42.86%?
    ReleaseIdentifier.java?91.78%?
    ReleaseIdentifier_Factory.java?0.00%?
    ReleaseUtils.java?83.33%?
    ScreenshotTaker.java?36.17%?
    ScreenshotTaker_Factory.java?0.00%?
    SequentialReference.java?100.00%?
    SignInResultActivity.java?0.00%?
    SignInStorage.java?100.00%?
    SignInStorage_Factory.java?0.00%?
    TakeScreenshotAndStartFeedbackActivity.java?0.00%?
    TakeScreenshotAndStartFeedbackActivity_MembersInjector.java?0.00%?
    TaskCache.java?100.00%?
    TaskCompletionSourceCache.java?72.41%?
    TaskUtils.java?77.50%?
    TesterApiDisabledErrorDetails.java?93.75%?
    TesterApiHttpClient.java?90.09%?
    TesterApiHttpClient_Factory.java?0.00%?
    TesterSignInManager.java?95.45%?
    TesterSignInManager_Factory.java?0.00%?
    UpdateProgressImpl.java?100.00%?
    UpdateTaskCache.java?91.30%?
    UpdateTaskImpl.java?76.32%?

  • firebase-common

    Overall coverage changed from 54.83% (250c69f) to 51.78% (109a2b5) by -3.06%.

    FilenameBase (250c69f)Merge (109a2b5)Diff
    Logger.kt?0.00%?
  • firebase-crashlytics

    Overall coverage changed from 9.62% (250c69f) to 9.55% (109a2b5) by -0.07%.

    FilenameBase (250c69f)Merge (109a2b5)Diff
    AutoCrashlyticsReportEncoder.java57.67%57.59%-0.07%
    AutoValue_CrashlyticsReport_Session_Event_Application_ProcessDetails.java?0.00%?
    CrashlyticsReport.java2.59%2.52%-0.07%
    CrashlyticsReportDataCapture.java3.45%3.35%-0.10%
    CrashlyticsReportJsonTransform.java1.17%1.11%-0.06%
    ProcessDetailsProvider.kt?0.00%?
  • firebase-database

    Overall coverage changed from 50.17% (250c69f) to 50.13% (109a2b5) by -0.04%.

    FilenameBase (250c69f)Merge (109a2b5)Diff
    ChildChangeAccumulator.java96.67%83.33%-13.33%
  • firebase-messaging

    Overall coverage changed from 83.84% (250c69f) to 83.87% (109a2b5) by +0.04%.

    FilenameBase (250c69f)Merge (109a2b5)Diff
    FirebaseMessaging.java73.57%73.13%-0.44%
    RequestDeduplicator.java80.00%90.00%+10.00%

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/ckTx5Gg13x.html

David Motsonashvili and others added 10 commits October 20, 2023 16:47
Per [b/306709509](https://b.corp.google.com/issues/306709509),

This fixes an oversight in which we were not updating the
`latestReleasedVersion` during our version bumps.

Additionally, this fixes the following:

- [b/306713770](https://b.corp.google.com/issues/306713770) -> Update
the `latestReleasedVersion` for libraries where it's invalid
- [b/300682493](https://b.corp.google.com/issues/300682493) -> Add
condition to `moveUnreleasedChanges` task for changelog file existence
Add a common Firebase logging interface to make logging more consistent
and make testing easier. With this we could make tests that assert
specific things were logged, for example, Sessions could assert an event
was logged.
Add process details to Crashlytics java and anr events. NDK events will
be handled later after we upgrade crashpad. Also fixed a bug in
detecting background processes.

The proto change was made in internal cl/574516195
@davidmotson
Copy link
Collaborator

/make bom

@github-actions
Copy link
Contributor Author

buildSrc Test Results

37 tests   37 ✔️  1m 26s ⏱️
  8 suites    0 💤
  8 files      0

Results for commit 1c9f06f.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants