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

Merging firebase-dataconnect to main #6288

Closed
wants to merge 636 commits into from
Closed

Conversation

dconeybe
Copy link
Contributor

No description provided.

…ists of Int64 and Timestamp, since b/335903750 has been fixed (#546)
…ad of apply(from = "../gradle/googleServices.gradle")
…ing sent, but should have been sent as null (#550)
…that fail due to lack of null list support in the backend (b/337312723) (#552)
…uth emulator, since the auth emulator startup is slow
Copy link
Contributor

github-actions bot commented Sep 20, 2024

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

@google-oss-bot
Copy link
Contributor

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

Generated by 🚫 Danger

Copy link
Contributor

Unit Test Results

   901 files  +   893     901 suites  +893   32m 49s ⏱️ + 32m 21s
4 851 tests +4 829  4 830 ✔️ +4 808  21 💤 +21  0 ±0 
9 760 runs  +9 716  9 718 ✔️ +9 674  42 💤 +42  0 ±0 

Results for commit 2110dae. ± Comparison against base commit da0eefa.

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • firebase-common

    TypeBase (da0eefa)Merge (2ef2464)Diff
    apk (release)4.40 MB4.45 MB+46.9 kB (+1.1%)
  • firebase-common-ktx

    TypeBase (da0eefa)Merge (2ef2464)Diff
    apk (release)4.40 MB4.45 MB+46.9 kB (+1.1%)
  • firebase-database

    TypeBase (da0eefa)Merge (2ef2464)Diff
    aar543 kB542 kB-41 B (-0.0%)
    apk (release)5.29 MB5.34 MB+47.1 kB (+0.9%)
  • firebase-database-ktx

    TypeBase (da0eefa)Merge (2ef2464)Diff
    apk (release)5.29 MB5.34 MB+47.1 kB (+0.9%)
  • firebase-dataconnect

    TypeBase (da0eefa)Merge (2ef2464)Diff
    aar?641 kB? (?)
    apk (aggressive)?520 kB? (?)
    apk (release)?11.0 MB? (?)
  • firebase-firestore

    TypeBase (da0eefa)Merge (2ef2464)Diff
    aar1.44 MB1.44 MB+404 B (+0.0%)
    apk (release)11.3 MB11.4 MB+47.1 kB (+0.4%)
  • firebase-firestore-ktx

    TypeBase (da0eefa)Merge (2ef2464)Diff
    apk (release)11.3 MB11.4 MB+47.1 kB (+0.4%)
  • firebase-storage

    TypeBase (da0eefa)Merge (2ef2464)Diff
    apk (release)4.99 MB5.04 MB+47.1 kB (+0.9%)
  • firebase-storage-ktx

    TypeBase (da0eefa)Merge (2ef2464)Diff
    apk (release)4.99 MB5.04 MB+47.1 kB (+0.9%)

Test Logs

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

@google-oss-bot
Copy link
Contributor

Coverage Report 1

Affected Products

  • firebase-config

    Overall coverage changed from ? (da0eefa) to 84.11% (2ef2464) by ?.

    33 individual files with coverage change

    FilenameBase (da0eefa)Merge (2ef2464)Diff
    AutoValue_ConfigUpdate.java?29.41%?
    Code.java?0.00%?
    ConfigAutoFetch.java?87.74%?
    ConfigCacheClient.java?93.33%?
    ConfigContainer.java?94.29%?
    ConfigFetchHandler.java?92.86%?
    ConfigFetchHttpClient.java?87.33%?
    ConfigGetParameterHandler.java?96.45%?
    ConfigMetadataClient.java?92.31%?
    ConfigRealtimeHandler.java?41.38%?
    ConfigRealtimeHttpClient.java?68.93%?
    ConfigStorageClient.java?100.00%?
    ConfigUpdate.java?100.00%?
    ConfigUpdateListener.java?0.00%?
    ConfigUpdateListenerRegistration.java?0.00%?
    DefaultsXmlParser.java?0.00%?
    FirebaseRemoteConfig.java?89.52%?
    FirebaseRemoteConfigClientException.java?75.00%?
    FirebaseRemoteConfigException.java?95.65%?
    FirebaseRemoteConfigFetchThrottledException.java?100.00%?
    FirebaseRemoteConfigInfo.java?0.00%?
    FirebaseRemoteConfigInfoImpl.java?100.00%?
    FirebaseRemoteConfigServerException.java?68.42%?
    FirebaseRemoteConfigSettings.java?61.54%?
    FirebaseRemoteConfigValue.java?0.00%?
    FirebaseRemoteConfigValueImpl.java?84.62%?
    Personalization.java?91.43%?
    RemoteConfig.kt?31.58%?
    RemoteConfigComponent.java?90.70%?
    RemoteConfigConstants.java?0.00%?
    RemoteConfigRegistrar.java?100.00%?
    RolloutsStateFactory.java?95.24%?
    RolloutsStateSubscriptionsHandler.java?100.00%?

  • firebase-database

    Overall coverage changed from 50.16% (da0eefa) to 50.21% (2ef2464) by +0.04%.

    FilenameBase (da0eefa)Merge (2ef2464)Diff
    ChildChangeAccumulator.java83.33%96.67%+13.33%
  • firebase-firestore

    Overall coverage changed from 46.06% (da0eefa) to 46.05% (2ef2464) by -0.01%.

    FilenameBase (da0eefa)Merge (2ef2464)Diff
    LruGarbageCollector.java97.27%93.64%-3.64%
    SetMutation.java94.44%97.22%+2.78%
  • firebase-messaging

    Overall coverage changed from 84.24% (da0eefa) to 84.28% (2ef2464) by +0.04%.

    FilenameBase (da0eefa)Merge (2ef2464)Diff
    FirebaseMessaging.java76.80%76.00%-0.80%
    Metadata.java36.51%41.27%+4.76%
  • firebase-ml-modeldownloader

    Overall coverage changed from 83.32% (da0eefa) to 83.38% (2ef2464) by +0.06%.

    FilenameBase (da0eefa)Merge (2ef2464)Diff
    FirebaseModelDownloader.java84.05%84.66%+0.61%
  • firebase-storage

    Overall coverage changed from 84.33% (da0eefa) to 83.96% (2ef2464) by -0.37%.

    FilenameBase (da0eefa)Merge (2ef2464)Diff
    StorageException.java69.09%65.45%-3.64%
    StorageTask.java83.99%83.38%-0.60%
    UploadTask.java83.17%81.52%-1.65%

Test Logs

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

@google-oss-bot
Copy link
Contributor

Startup Time Report 1

The report is too large (122,652 chars) to be displayed on GitHub. Please check this report on GCS.

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

@dconeybe dconeybe closed this Sep 21, 2024
@dconeybe dconeybe deleted the dataconnect-merge_to_main branch September 21, 2024 01:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants