Skip to content

Conversation

daymxn
Copy link
Member

@daymxn daymxn commented Feb 24, 2025

Per b/398840288,

This bumps protolite-well-known-types to properly utilize 3.25.5. It seems as though this was an oversight in #6343, but since gradle uses the highest version when resolving dependency conflicts (and all the existing libraries already use 3.25.5), this isn't a major issue. This is only really an issue if someone is using protolite-well-known-types in isolation (which isn't really a use-case we're shipping for). But the main reason for fixing this is that it causes a bit of confusion when trying to track dependency issues (see issue #6674 for an example of this).

Fixes #6674

Copy link
Contributor

github-actions bot commented Feb 24, 2025

Release note changes

The following release notes were modified. Please ensure they look correct.

Release Notes
firebase-firestore
### {{firestore}} version 25.1.3 {: #firestore_v25-1-3} * {{changed}} Updated `protolite-well-known-types` dependency to `18.0.1`. GitHub [#6716](//github.com/firebase/firebase-android-sdk/issues/6716){: .external} #### {{firestore}} Kotlin extensions version 25.1.3 {: #firestore-ktx_v25-1-3} The Kotlin extensions library transitively includes the updated `firebase-firestore` library. The Kotlin extensions library has no additional updates. 
firebase-inappmessaging-display
### {{inappmessaging}} Display version 21.0.2 {: #inappmessaging-display_v21-0-2} * {{changed}} Updated `protolite-well-known-types` dependency to `18.0.1`. GitHub [#6716](//github.com/firebase/firebase-android-sdk/issues/6716){: .external} #### {{inappmessaging}} Display Kotlin extensions version 21.0.2 {: #inappmessaging-display-ktx_v21-0-2} The Kotlin extensions library transitively includes the updated `firebase-inappmessaging-display` library. The Kotlin extensions library has no additional updates. 
firebase-inappmessaging
### {{inappmessaging}} version 21.0.2 {: #inappmessaging_v21-0-2} * {{changed}} Updated `protolite-well-known-types` dependency to `18.0.1`. GitHub [#6716](//github.com/firebase/firebase-android-sdk/issues/6716){: .external} #### {{inappmessaging}} Kotlin extensions version 21.0.2 {: #inappmessaging-ktx_v21-0-2} The Kotlin extensions library transitively includes the updated `firebase-inappmessaging` library. The Kotlin extensions library has no additional updates. 
firebase-perf
### {{perfmon}} version 21.0.5 {: #performance_v21-0-5} * {{changed}} Updated `protolite-well-known-types` dependency to `18.0.1`. GitHub [#6716](//github.com/firebase/firebase-android-sdk/issues/6716){: .external} #### {{perfmon}} Kotlin extensions version 21.0.5 {: #performance-ktx_v21-0-5} The Kotlin extensions library transitively includes the updated `firebase-perf` library. The Kotlin extensions library has no additional updates. 

The following had changelogs that were modified, but did not have any unreleased entries for release notes to generate from.

Changelogs

protolite-well-known-types

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 24, 2025

Coverage Report 1

This report is too large (93,471 characters) to be displayed here in a GitHub comment. Please use the below link to see the full report on Google Cloud Storage.

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/eGOn5lBAjJ.html
Copy link
Contributor

github-actions bot commented Feb 24, 2025

Test Results

  360 files  +  340    360 suites  +340   9m 2s ⏱️ + 8m 50s
2 522 tests +2 406  2 505 ✅ +2 389  17 💤 +17  0 ❌ ±0 
5 080 runs  +4 848  5 046 ✅ +4 814  34 💤 +34  0 ❌ ±0 

Results for commit b7c6043. ± Comparison against base commit 22be120.

This pull request removes 116 and adds 2522 tests. Note that renamed tests count towards both.
com.google.firebase.vertexai.GenerativeModelTesting ‑ exception thrown when using invalid location com.google.firebase.vertexai.GenerativeModelTesting ‑ system calling in request com.google.firebase.vertexai.SchemaTests ‑ basic schema declaration com.google.firebase.vertexai.SchemaTests ‑ full schema declaration com.google.firebase.vertexai.StreamingSnapshotTests ‑ citation parsed correctly com.google.firebase.vertexai.StreamingSnapshotTests ‑ empty content com.google.firebase.vertexai.StreamingSnapshotTests ‑ http errors com.google.firebase.vertexai.StreamingSnapshotTests ‑ image rejected com.google.firebase.vertexai.StreamingSnapshotTests ‑ invalid api key com.google.firebase.vertexai.StreamingSnapshotTests ‑ invalid json … 
com.google.firebase.firestore.AggregateQuerySnapshotTest ‑ createWithCountShouldReturnInstanceWithTheGivenQueryAndCount com.google.firebase.firestore.AggregateQueryTest ‑ testSourceMustNotBeNull com.google.firebase.firestore.BlobTest ‑ testComparison com.google.firebase.firestore.BlobTest ‑ testEquals com.google.firebase.firestore.BlobTest ‑ testMutableBytes com.google.firebase.firestore.CollectionReferenceTest ‑ testEquals com.google.firebase.firestore.DocumentChangeTest ‑ randomTests com.google.firebase.firestore.DocumentChangeTest ‑ testAdditions com.google.firebase.firestore.DocumentChangeTest ‑ testChangesWithSortOrderChange com.google.firebase.firestore.DocumentChangeTest ‑ testDeletions … 

♻️ This comment has been updated with latest results.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 24, 2025

Size Report 1

Affected Products

  • firebase-firestore

    TypeBase (22be120)Merge (85d6fe5)Diff
    apk (release)11.4 MB11.4 MB+2.39 kB (+0.0%)
  • firebase-inappmessaging

    TypeBase (22be120)Merge (85d6fe5)Diff
    apk (aggressive)1.05 MB1.05 MB+588 B (+0.1%)
    apk (release)11.7 MB11.7 MB+2.85 kB (+0.0%)
  • firebase-inappmessaging-display

    TypeBase (22be120)Merge (85d6fe5)Diff
    apk (aggressive)2.24 MB2.24 MB-528 B (-0.0%)
    apk (release)15.9 MB15.9 MB+3.17 kB (+0.0%)
  • firebase-perf

    TypeBase (22be120)Merge (85d6fe5)Diff
    apk (release)10.1 MB10.1 MB+2.93 kB (+0.0%)
  • protolite-well-known-types

    TypeBase (22be120)Merge (85d6fe5)Diff
    aar1.01 MB1.02 MB+2.21 kB (+0.2%)
    apk (aggressive)134 kB134 kB+144 B (+0.1%)
    apk (release)1.68 MB1.71 MB+32.0 kB (+1.9%)

Test Logs

  1. https://storage.googleapis.com/firebase-sdk-metric-reports/MIHLKsoDgC.html
@daymxn daymxn requested a review from rlazo February 24, 2025 17:08
@daymxn daymxn self-assigned this Feb 24, 2025
@daymxn daymxn requested a review from rlazo February 24, 2025 18:42
@daymxn daymxn merged commit 4cf2825 into main Feb 24, 2025
55 of 56 checks passed
@daymxn daymxn deleted the bump-protolite branch February 24, 2025 20:37
@firebase firebase locked and limited conversation to collaborators Mar 27, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

3 participants