Skip to content

Edit java docs for Firebase App Distribution #3393

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 4 commits into from
Feb 3, 2022

Conversation

rachaprince
Copy link

@rachaprince rachaprince commented Feb 2, 2022

Updates Java doc comments based on code review feedback on the auto generated java docs.

Once this merges into the main branch, the test release branch will automatically re-generate the zip file with the docs.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 2, 2022

Coverage Report 1

Affected Products

No changes between base commit (bce1a75) and merge commit (3904713).

Test Logs

Notes

  • Commit (3904713) is created by Prow via merging PR base commit (bce1a75) and head commit (0de60ad).
  • 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/nnPdOr54dk.html

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Feb 2, 2022

Size Report 1

Affected Products

  • firebase-appdistribution

    TypeBase (bce1a75)Merge (3904713)Diff
    aar123 kB123 kB-9 B (-0.0%)
    apk (release)1.55 MB1.55 MB-8 B (-0.0%)

Test Logs

Notes

  • Commit (3904713) is created by Prow via merging PR base commit (bce1a75) and head commit (0de60ad).

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

@rachaprince rachaprince requested a review from lfkellogg February 3, 2022 15:59
Copy link
Contributor

@lfkellogg lfkellogg left a comment

Choose a reason for hiding this comment

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

A few nits but this looks great!

* release is available.
* The number of bytes downloaded so far for an APK.
*
* @returns the number of bytes downloaded, or -1 if called when updating to an AAB or if no new
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, and this should be @return not @returns

* release is available.
* The file size of the APK file to download in bytes.
*
* @returns the file size in bytes, or -1 if called when updating to an AAB or if no new release
Copy link
Contributor

Choose a reason for hiding this comment

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

Oh, and this should be @return not @returns

@rachaprince rachaprince merged commit ccd1cb6 into master Feb 3, 2022
@rachaprince rachaprince deleted the fad-update-java-doc-comments branch February 3, 2022 17:26
@rachaprince rachaprince restored the fad-update-java-doc-comments branch February 3, 2022 18:34
@rachaprince rachaprince deleted the fad-update-java-doc-comments branch February 3, 2022 18:34
@firebase firebase locked and limited conversation to collaborators Mar 6, 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.

4 participants