Skip to content

update kotlin from 1.3.72 to 1.4.32 #3855

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 6 commits into from
Jul 26, 2022

Conversation

thatfiredev
Copy link
Member

@thatfiredev thatfiredev commented Jun 29, 2022

Since we can't yet upgrade to Kotlin 1.6 or later versions, I'm making small incremental changes to get us there.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 29, 2022

Coverage Report 1

Affected Products

  • firebase-database

    Overall coverage changed from 50.23% (1737a87) to 50.16% (62e7c4a) by -0.07%.

    FilenameBase (1737a87)Merge (62e7c4a)Diff
    ChildChangeAccumulator.java96.67%83.33%-13.33%
    DoubleNode.java100.00%88.24%-11.76%
  • firebase-firestore

    Overall coverage changed from 45.92% (1737a87) to 45.91% (62e7c4a) by -0.01%.

    FilenameBase (1737a87)Merge (62e7c4a)Diff
    DeleteMutation.java95.24%90.48%-4.76%
    PatchMutation.java100.00%98.39%-1.61%
  • firebase-installations

    Overall coverage changed from ? (1737a87) to 60.10% (62e7c4a) by ?.

    22 individual files with coverage change

    FilenameBase (1737a87)Merge (62e7c4a)Diff
    AutoValue_InstallationResponse.java?58.93%?
    AutoValue_PersistedInstallationEntry.java?61.18%?
    AutoValue_TokenResult.java?45.65%?
    AwaitListener.java?0.00%?
    Clock.java?0.00%?
    CrossProcessLock.java?51.85%?
    FirebaseInstallations.java?98.05%?
    FirebaseInstallationServiceClient.java?4.80%?
    FirebaseInstallationsException.java?78.57%?
    FirebaseInstallationsRegistrar.java?100.00%?
    GetAuthTokenListener.java?100.00%?
    GetIdListener.java?70.00%?
    IidStore.java?44.78%?
    InstallationResponse.java?100.00%?
    PersistedInstallation.java?97.01%?
    PersistedInstallationEntry.java?100.00%?
    RandomFidGenerator.java?20.00%?
    RequestLimiter.java?100.00%?
    StateListener.java?0.00%?
    SystemClock.java?100.00%?
    TokenResult.java?100.00%?
    Utils.java?95.00%?

  • firebase-messaging

    Overall coverage changed from 85.03% (1737a87) to 85.11% (62e7c4a) by +0.08%.

    FilenameBase (1737a87)Merge (62e7c4a)Diff
    MessagingAnalytics.java81.38%82.19%+0.81%

Test Logs

Notes

  • Commit (62e7c4a) is created by Prow via merging PR base commit (1737a87) and head commit (0ad18c8).
  • Run gradle <product>:checkCoverage to produce HTML coverage reports locally. After gradle commands finished, report files can be found under <product-build-dir>/reports/jacoco/.

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

@thatfiredev
Copy link
Member Author

thatfiredev commented Jun 29, 2022

/retest

1 similar comment
@thatfiredev
Copy link
Member Author

/retest

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Jun 29, 2022

Size Report 1

Affected Products

  • firebase-appcheck-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar5.59 kB5.60 kB+5 B (+0.1%)
    apk (aggressive)350 kB349 kB-953 B (-0.3%)
    apk (release)1.47 MB1.53 MB+63.6 kB (+4.3%)
  • firebase-appdistribution-api-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar6.13 kB6.14 kB+5 B (+0.1%)
    apk (aggressive)101 kB101 kB-937 B (-0.9%)
    apk (release)1.18 MB1.24 MB+65.3 kB (+5.6%)
  • firebase-common-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar6.28 kB6.34 kB+55 B (+0.9%)
    apk (aggressive)100 kB99.3 kB-1.22 kB (-1.2%)
    apk (release)1.17 MB1.22 MB+55.9 kB (+4.8%)
  • firebase-config-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar5.85 kB5.86 kB+10 B (+0.2%)
    apk (aggressive)114 kB113 kB-945 B (-0.8%)
    apk (release)1.22 MB1.29 MB+66.7 kB (+5.5%)
  • firebase-crashlytics-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar6.85 kB6.87 kB+13 B (+0.2%)
    apk (aggressive)232 kB231 kB-921 B (-0.4%)
    apk (release)1.38 MB1.44 MB+62.7 kB (+4.5%)
  • firebase-database-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar7.71 kB7.67 kB-36 B (-0.5%)
    apk (aggressive)349 kB348 kB-937 B (-0.3%)
    apk (release)1.63 MB1.69 MB+64.3 kB (+4.0%)
  • firebase-dynamic-links-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar8.79 kB8.80 kB+9 B (+0.1%)
    apk (aggressive)349 kB348 kB-901 B (-0.3%)
    apk (release)1.47 MB1.53 MB+65.0 kB (+4.4%)
  • firebase-firestore-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar7.47 kB7.42 kB-45 B (-0.6%)
    apk (aggressive)505 kB504 kB-941 B (-0.2%)
    apk (release)3.87 MB3.94 MB+65.8 kB (+1.7%)
  • firebase-functions-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar6.14 kB6.15 kB+9 B (+0.1%)
    apk (aggressive)413 kB412 kB-961 B (-0.2%)
    apk (release)1.70 MB1.76 MB+65.2 kB (+3.8%)
  • firebase-inappmessaging-display-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar22.2 kB22.2 kB+11 B (+0.0%)
    apk (aggressive)1.51 MB1.51 MB-937 B (-0.1%)
    apk (release)5.18 MB5.24 MB+62.5 kB (+1.2%)
  • firebase-inappmessaging-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar5.29 kB5.30 kB+7 B (+0.1%)
    apk (aggressive)680 kB679 kB-929 B (-0.1%)
    apk (release)3.91 MB3.97 MB+64.6 kB (+1.7%)
  • firebase-installations-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar5.28 kB5.29 kB+3 B (+0.1%)
    apk (aggressive)102 kB101 kB-917 B (-0.9%)
    apk (release)1.19 MB1.26 MB+63.5 kB (+5.3%)
  • firebase-messaging-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar6.53 kB6.55 kB+16 B (+0.2%)
    apk (aggressive)454 kB453 kB-917 B (-0.2%)
    apk (release)1.63 MB1.69 MB+66.6 kB (+4.1%)
  • firebase-ml-modeldownloader-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar6.40 kB6.41 kB+14 B (+0.2%)
    apk (aggressive)159 kB158 kB-901 B (-0.6%)
    apk (release)1.31 MB1.37 MB+63.4 kB (+4.8%)
  • firebase-perf-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar19.5 kB19.5 kB+28 B (+0.1%)
    apk (aggressive)1.04 MB1.04 MB-961 B (-0.1%)
    apk (release)2.96 MB3.02 MB+61.8 kB (+2.1%)
  • firebase-storage-ktx

    TypeBase (1737a87)Merge (62e7c4a)Diff
    aar7.29 kB7.30 kB+8 B (+0.1%)
    apk (aggressive)348 kB347 kB-933 B (-0.3%)
    apk (release)1.50 MB1.56 MB+63.3 kB (+4.2%)

Test Logs

Notes

  • Commit (62e7c4a) is created by Prow via merging PR base commit (1737a87) and head commit (0ad18c8).

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

@thatfiredev
Copy link
Member Author

/test check-changed

@rlazo rlazo self-requested a review July 25, 2022 17:30
Copy link
Collaborator

@rlazo rlazo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Kotlindoc still works after this change.

@thatfiredev
Copy link
Member Author

/test check-changed

@rlazo rlazo merged commit 4bf252c into firebase:master Jul 26, 2022
lfkellogg pushed a commit that referenced this pull request Aug 5, 2022
* update kotlin from 1.3.72 to 1.4.32

* use compileOnly in tools/lint/lint.gradle

* update dagger version in protoc-gen-firebase-encoders

* stop using kotlin-stdlib-jdk7 in firebase-common

* filter out only release component
@firebase firebase locked and limited conversation to collaborators Aug 26, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants