Skip to content

Conversation

@tgillam-do
Copy link
Collaborator

This retrieves the ADK version from the correct metadata and ensures that it gets sent to the deploy service correctly. The previous version could produce wrong result.

@tgillam-do tgillam-do requested a review from Copilot January 9, 2026 22:02
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the method for retrieving the ADK version to use package metadata instead of accessing a module-level attribute. This change improves reliability by querying the installed package metadata directly, preventing potential issues where the module attribute might be incorrect or unavailable.

Changes:

  • Replaced direct gradient_adk.__version__ access with a new _get_adk_version() helper function
  • Added importlib.metadata import to retrieve version from package metadata
  • Implemented graceful fallback to "unknown" when package metadata is unavailable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@tgillam-do tgillam-do merged commit 2c6a401 into main Jan 9, 2026
5 checks passed
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.

3 participants