Skip to content

chore: cap eval Vertex AI dependency below 2 - #7016

Open
UniversePeak wants to merge 1 commit into
google:mainfrom
UniversePeak:fix/eval-aiplatform-upper-bound
Open

chore: cap eval Vertex AI dependency below 2#7016
UniversePeak wants to merge 1 commit into
google:mainfrom
UniversePeak:fix/eval-aiplatform-upper-bound

Conversation

@UniversePeak

Copy link
Copy Markdown

Link to Issue or Description of Change

Problem:
The eval optional dependency allowed google-cloud-aiplatform 2.x, while the all, gcp, and test extras already constrain the same dependency below 2. Users installing google-adk[eval] could therefore resolve a different major line from the sibling extras.

Solution:
Add the <2 upper bound to google-cloud-aiplatform[evaluation] in the eval extra. Add a release-dependency regression test that verifies the lower bound remains admitted and the 2.x line is excluded.

Testing Plan

Unit Tests:

  • python -m pytest tests/unittests/test_release_dependencies.py -q -k eval_extra_caps_google_cloud_aiplatform_at_v2 (1 passed)
  • python -m compileall -q tests/unittests/test_release_dependencies.py
  • git diff --check

The remaining tests in tests/unittests/test_release_dependencies.py require the repository's installed ADK dependencies; the focused dependency-contract test passes in this checkout.

@google-cla

google-cla Bot commented Sep 4, 2026

Copy link
Copy Markdown

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@UniversePeak
UniversePeak force-pushed the fix/eval-aiplatform-upper-bound branch from 084a167 to 4d19d12 Compare September 4, 2026 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants