Skip to content

Conversation

@mrichards
Copy link
Contributor

This is not a behavior change; it just clarifies the existing behavior in the javadoc.

@mrichards mrichards requested review from cbonnie and mrwillis21 April 23, 2020 15:29
@googlebot googlebot added the cla: yes Override cla label Apr 23, 2020
@google-oss-bot
Copy link
Contributor

Coverage Report

Affected SDKs

No changes between base commit (0c89bff) and head commit (1409294).

Test Logs

Notes

HTML coverage reports can be produced locally with ./gradlew <product>:checkCoverage.
Report files are located at <product-build-dir>/reports/jacoco/.

@google-oss-bot
Copy link
Contributor

Binary Size Report

Affected SDKs

  • firebase-common

    Type Base (0c89bff) Head (1409294) Diff
    apk (aggressive) 82.5 kB 82.5 kB -11 B (-0.0%)
    apk (debug) 770 kB 770 kB -34 B (-0.0%)
  • firebase-components

    Type Base (0c89bff) Head (1409294) Diff
    apk (aggressive) 11.0 kB 10.9 kB -17 B (-0.2%)
    apk (debug) 35.9 kB 35.9 kB +3 B (+0.0%)
  • firebase-crashlytics

    Type Base (0c89bff) Head (1409294) Diff
    aar 400 kB 400 kB +4 B (+0.0%)
    apk (aggressive) 598 kB 598 kB +4 B (+0.0%)
    apk (debug) 1.60 MB 1.60 MB -42 B (-0.0%)
    apk (release) 1.36 MB 1.36 MB +10 B (+0.0%)
  • firebase-encoders-json

    Type Base (0c89bff) Head (1409294) Diff
    apk (aggressive) 11.0 kB 10.9 kB -17 B (-0.2%)
    apk (debug) 27.4 kB 27.4 kB +1 B (+0.0%)
  • transport-api

    Type Base (0c89bff) Head (1409294) Diff
    apk (aggressive) 11.0 kB 10.9 kB -17 B (-0.2%)
    apk (debug) 23.0 kB 23.0 kB -13 B (-0.1%)
  • transport-backend-cct

    Type Base (0c89bff) Head (1409294) Diff
    apk (aggressive) 47.9 kB 47.9 kB +4 B (+0.0%)
    apk (debug) 101 kB 101 kB +2 B (+0.0%)
  • transport-runtime

    Type Base (0c89bff) Head (1409294) Diff
    apk (aggressive) 35.6 kB 35.5 kB -11 B (-0.0%)
    apk (debug) 80.5 kB 80.5 kB +19 B (+0.0%)

Test Logs

@google-oss-bot
Copy link
Contributor

@mrichards: The following test failed, say /retest to rerun them all:

Test name Commit Details Rerun command
smoke-tests 1409294 link /test smoke-tests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

* sending them to Crashlytics.
*
* @param enabled whether to enable automatic data collection.
* @param enabled whether to enable automatic data collection. When set to `false`, the new value
Copy link

Choose a reason for hiding this comment

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

Suggest:

* @param specifies whether to enable automatic data collection...

@mrichards mrichards merged commit 9dae188 into master Apr 24, 2020
@mrichards mrichards deleted the crashlytics-javadoc-update branch April 24, 2020 17:58
@firebase firebase locked and limited conversation to collaborators May 25, 2020
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.

5 participants