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

Refactor Vendor Plugin #6640

Open
wants to merge 6 commits into
base: main
Choose a base branch
from
Open

Refactor Vendor Plugin #6640

wants to merge 6 commits into from

Conversation

daymxn
Copy link
Member

@daymxn daymxn commented Jan 21, 2025

Per b/391351857,

This refactors the VendorPlugin and VendorTask to be more compliant with the gradle cache and configuration avoidance. It also adds proper task logging and documentation to both the task and the plugin.

This PR also fixes the following:

  • b/391352175 -> Rename plugin Util files to Extensions

Copy link
Contributor

github-actions bot commented Jan 21, 2025

📝 PRs merging into main branch

Our main branch should always be in a releasable state. If you are working on a larger change, or if you don't want this change to see the light of the day just yet, consider using a feature branch first, and only merge into the main branch when the code complete and ready to be released.

Copy link
Contributor

Vertex AI Mock Responses Check ⚠️

A newer major version of the mock responses for Vertex AI unit tests is available. update_responses.sh should be updated to clone the latest version of the responses: v5.3

@daymxn daymxn enabled auto-merge (squash) January 21, 2025 16:39
Copy link
Contributor

plugins test results

37 tests   37 ✅  2m 7s ⏱️
 8 suites   0 💤
 8 files     0 ❌

Results for commit ea89b60.

Copy link
Contributor

Test Results

 1 034 files  + 1 014   1 034 suites  +1 014   34m 17s ⏱️ + 34m 5s
 5 888 tests + 5 775   5 866 ✅ + 5 753  22 💤 +22  0 ❌ ±0 
11 839 runs  +11 613  11 795 ✅ +11 569  44 💤 +44  0 ❌ ±0 

Results for commit ea89b60. ± Comparison against base commit cf5fe2e.

@google-oss-bot
Copy link
Contributor

Size Report 1

Affected Products

  • base

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    apk (aggressive)?8.80 kB? (?)
    apk (release)?9.77 kB? (?)
  • firebase-abt

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?12.4 kB? (?)
    apk (aggressive)?141 kB? (?)
    apk (release)?4.41 MB? (?)
  • firebase-annotations

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    apk (aggressive)?8.80 kB? (?)
    apk (release)?12.0 kB? (?)
  • firebase-appcheck

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?39.2 kB? (?)
    apk (aggressive)?390 kB? (?)
    apk (release)?4.88 MB? (?)
  • firebase-appcheck-debug

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?9.34 kB? (?)
    apk (aggressive)?396 kB? (?)
    apk (release)?4.89 MB? (?)
  • firebase-appcheck-debug-testing

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?5.39 kB? (?)
    apk (aggressive)?421 kB? (?)
    apk (release)?5.05 MB? (?)
  • firebase-appcheck-interop

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.13 kB? (?)
    apk (aggressive)?318 kB? (?)
    apk (release)?1.71 MB? (?)
  • firebase-appcheck-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.52 kB? (?)
    apk (aggressive)?390 kB? (?)
    apk (release)?4.88 MB? (?)
  • firebase-appcheck-playintegrity

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?9.12 kB? (?)
    apk (aggressive)?405 kB? (?)
    apk (release)?4.93 MB? (?)
  • firebase-appdistribution

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?230 kB? (?)
    apk (aggressive)?1.23 MB? (?)
    apk (release)?7.84 MB? (?)
  • firebase-appdistribution-api

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?18.9 kB? (?)
    apk (aggressive)?145 kB? (?)
    apk (release)?4.42 MB? (?)
  • firebase-appdistribution-api-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.58 kB? (?)
    apk (aggressive)?145 kB? (?)
    apk (release)?4.42 MB? (?)
  • firebase-common

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?116 kB? (?)
    apk (aggressive)?140 kB? (?)
    apk (release)?4.45 MB? (?)
  • firebase-common-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.18 kB? (?)
    apk (aggressive)?141 kB? (?)
    apk (release)?4.45 MB? (?)
  • firebase-components

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?45.7 kB? (?)
    apk (aggressive)?20.8 kB? (?)
    apk (release)?2.06 MB? (?)
  • firebase-config

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?112 kB? (?)
    apk (aggressive)?211 kB? (?)
    apk (release)?4.59 MB? (?)
  • firebase-config-interop

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?10.5 kB? (?)
    apk (aggressive)?20.8 kB? (?)
    apk (release)?2.02 MB? (?)
  • firebase-config-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.26 kB? (?)
    apk (aggressive)?212 kB? (?)
    apk (release)?4.59 MB? (?)
  • firebase-crashlytics

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?415 kB? (?)
    apk (aggressive)?699 kB? (?)
    apk (release)?5.85 MB? (?)
  • firebase-crashlytics-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.54 kB? (?)
    apk (aggressive)?699 kB? (?)
    apk (release)?5.85 MB? (?)
  • firebase-crashlytics-ndk

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?1.65 MB? (?)
    apk (aggressive / arm64-v8a)?1.87 MB? (?)
    apk (aggressive / armeabi-v7a)?1.35 MB? (?)
    apk (aggressive / x86)?1.85 MB? (?)
    apk (aggressive / x86_64)?1.92 MB? (?)
    apk (release / arm64-v8a)?7.02 MB? (?)
    apk (release / armeabi-v7a)?6.49 MB? (?)
    apk (release / x86)?7.00 MB? (?)
    apk (release / x86_64)?7.06 MB? (?)
  • firebase-database

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?542 kB? (?)
    apk (aggressive)?390 kB? (?)
    apk (release)?5.34 MB? (?)
  • firebase-database-collection

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?32.0 kB? (?)
    apk (aggressive)?318 kB? (?)
    apk (release)?1.86 MB? (?)
  • firebase-database-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.24 kB? (?)
    apk (aggressive)?391 kB? (?)
    apk (release)?5.34 MB? (?)
  • firebase-dataconnect

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?709 kB? (?)
    apk (aggressive)?440 kB? (?)
    apk (release)?10.0 MB? (?)
  • firebase-datatransport

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?4.28 kB? (?)
    apk (aggressive)?226 kB? (?)
    apk (release)?4.62 MB? (?)
  • firebase-decoders-json

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?7.85 kB? (?)
    apk (aggressive)?20.6 kB? (?)
    apk (release)?2.01 MB? (?)
  • firebase-dynamic-links

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?62.3 kB? (?)
    apk (aggressive)?387 kB? (?)
    apk (release)?4.89 MB? (?)
  • firebase-dynamic-links-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.61 kB? (?)
    apk (aggressive)?391 kB? (?)
    apk (release)?4.89 MB? (?)
  • firebase-dynamic-module-support

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?4.17 kB? (?)
    apk (aggressive)?157 kB? (?)
    apk (release)?4.52 MB? (?)
  • firebase-encoders

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    apk (aggressive)?20.6 kB? (?)
    apk (release)?2.01 MB? (?)
  • firebase-encoders-json

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?11.5 kB? (?)
    apk (aggressive)?20.8 kB? (?)
    apk (release)?2.02 MB? (?)
  • firebase-encoders-proto

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    apk (aggressive)?9.56 kB? (?)
    apk (release)?40.5 kB? (?)
  • firebase-encoders-reflective

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?7.97 kB? (?)
    apk (aggressive)?21.0 kB? (?)
    apk (release)?2.03 MB? (?)
  • firebase-firestore

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?1.45 MB? (?)
    apk (aggressive)?556 kB? (?)
    apk (release)?11.4 MB? (?)
  • firebase-firestore-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.23 kB? (?)
    apk (aggressive)?556 kB? (?)
    apk (release)?11.4 MB? (?)
  • firebase-functions

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?65.9 kB? (?)
    apk (aggressive)?439 kB? (?)
    apk (release)?5.44 MB? (?)
  • firebase-functions-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.24 kB? (?)
    apk (aggressive)?440 kB? (?)
    apk (release)?5.44 MB? (?)
  • firebase-inappmessaging

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?461 kB? (?)
    apk (aggressive)?1.05 MB? (?)
    apk (release)?11.7 MB? (?)
  • firebase-inappmessaging-display

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?163 kB? (?)
    apk (aggressive)?2.24 MB? (?)
    apk (release)?15.9 MB? (?)
  • firebase-inappmessaging-display-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.36 kB? (?)
    apk (aggressive)?2.24 MB? (?)
    apk (release)?15.9 MB? (?)
  • firebase-inappmessaging-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.27 kB? (?)
    apk (aggressive)?1.05 MB? (?)
    apk (release)?11.7 MB? (?)
  • firebase-installations

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?57.6 kB? (?)
    apk (aggressive)?148 kB? (?)
    apk (release)?4.46 MB? (?)
  • firebase-installations-interop

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?6.63 kB? (?)
    apk (aggressive)?70.9 kB? (?)
    apk (release)?1.27 MB? (?)
  • firebase-installations-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.20 kB? (?)
    apk (aggressive)?148 kB? (?)
    apk (release)?4.46 MB? (?)
  • firebase-messaging

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?149 kB? (?)
    apk (aggressive)?578 kB? (?)
    apk (release)?5.31 MB? (?)
  • firebase-messaging-directboot

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.20 kB? (?)
    apk (aggressive)?578 kB? (?)
    apk (release)?5.31 MB? (?)
  • firebase-messaging-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.54 kB? (?)
    apk (aggressive)?578 kB? (?)
    apk (release)?5.31 MB? (?)
  • firebase-ml-modeldownloader

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?125 kB? (?)
    apk (aggressive)?235 kB? (?)
    apk (release)?4.78 MB? (?)
  • firebase-ml-modeldownloader-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.60 kB? (?)
    apk (aggressive)?235 kB? (?)
    apk (release)?4.78 MB? (?)
  • firebase-perf

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?316 kB? (?)
    apk (aggressive)?1.63 MB? (?)
    apk (release)?10.1 MB? (?)
  • firebase-perf-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.21 kB? (?)
    apk (aggressive)?1.63 MB? (?)
    apk (release)?10.1 MB? (?)
  • firebase-sessions

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?168 kB? (?)
    apk (aggressive)?559 kB? (?)
    apk (release)?5.52 MB? (?)
  • firebase-storage

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?134 kB? (?)
    apk (aggressive)?398 kB? (?)
    apk (release)?5.04 MB? (?)
  • firebase-storage-ktx

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?3.24 kB? (?)
    apk (aggressive)?399 kB? (?)
    apk (release)?5.04 MB? (?)
  • firebase-vertexai

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?477 kB? (?)
    apk (aggressive)?1.09 MB? (?)
    apk (release)?9.15 MB? (?)
  • protolite-well-known-types

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?1.01 MB? (?)
    apk (aggressive)?134 kB? (?)
    apk (release)?1.68 MB? (?)
  • transport-api

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?12.1 kB? (?)
    apk (aggressive)?9.38 kB? (?)
    apk (release)?31.4 kB? (?)
  • transport-backend-cct

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?72.9 kB? (?)
    apk (aggressive)?97.8 kB? (?)
    apk (release)?258 kB? (?)
  • transport-runtime

    TypeBase (cf5fe2e)Merge (12e0844)Diff
    aar?163 kB? (?)
    apk (aggressive)?55.7 kB? (?)
    apk (release)?193 kB? (?)

Test Logs

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

@google-oss-bot
Copy link
Contributor

Coverage Report 1

This report is too large (235,951 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/BbgnGcC8mU.html

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.

2 participants