Skip to content

Fad logs file provider #2999

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 13 commits into from
Sep 23, 2021
Merged

Fad logs file provider #2999

merged 13 commits into from
Sep 23, 2021

Conversation

manny-jimenez
Copy link
Contributor

Correcting bug where the fileprovider name did not match with what the manifest had.

@manny-jimenez manny-jimenez self-assigned this Sep 23, 2021
@google-cla google-cla bot added the cla: yes Override cla label Sep 23, 2021
@google-oss-bot
Copy link
Contributor

google-oss-bot commented Sep 23, 2021

Coverage Report

Affected SDKs

  • firebase-app-distribution

    SDK overall coverage changed from 69.46% (b0a78ba) to 69.33% (612cc29e) by -0.13%.

    Filename Base (b0a78ba) Head (612cc29e) Diff
    CheckForNewReleaseClient.java 79.07% 79.55% +0.48%
    ReleaseIdentificationUtils.java 10.00% 9.43% -0.57%

Test Logs

Notes

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

Head commit (612cc29e) is created by Prow via merging commits: b0a78ba d640b23.

@google-oss-bot
Copy link
Contributor

google-oss-bot commented Sep 23, 2021

Binary Size Report

Affected SDKs

  • firebase-app-distribution

    Type Base (b0a78ba) Head (612cc29e) Diff
    aar 111 kB 111 kB +260 B (+0.2%)
    apk (aggressive) 739 kB 739 kB -24 B (-0.0%)
    apk (release) 1.54 MB 1.54 MB -8 B (-0.0%)

Test Logs

Notes

Head commit (612cc29e) is created by Prow via merging commits: b0a78ba d640b23.

@manny-jimenez manny-jimenez merged commit 5f29bb7 into master Sep 23, 2021
@manny-jimenez manny-jimenez deleted the FAD_logs_file_provider branch September 23, 2021 20:46
@google-oss-bot
Copy link
Contributor

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

Test name Commit Details Rerun command
smoke-tests d640b23 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.

@firebase firebase locked and limited conversation to collaborators Oct 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Override cla size/M
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants