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

Bump well known types #6716

Merged
merged 6 commits into from
Feb 24, 2025
Merged

Bump well known types #6716

merged 6 commits into from
Feb 24, 2025

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
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.

vulnerabilities because of javalite version 3.14.0 in firebase-perf
3 participants