Skip to content

Conversation

@bobhancockg
Copy link
Contributor

This change adds type annotations to all Python files within the examples/shopping_ads directory.

Type hints were added for function arguments, return values, class attributes, and local variables to improve code clarity, maintainability, and enable static analysis.

@bobhancockg bobhancockg requested a review from a team as a code owner May 24, 2025 13:48
@BenRKarl BenRKarl added the kokoro:run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
@kokoro-team kokoro-team removed the kokoro:run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
@BenRKarl BenRKarl added the kokoro:force-run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 5, 2025
@kokoro-team kokoro-team removed the kokoro:force-run This label is required to run a kokoro presubmit if an external contributor sends a PR. label Jun 6, 2025
This change adds type annotations to all Python files within the
examples/shopping_ads directory.

Type hints were added for function arguments, return values, class
attributes, and local variables to improve code clarity, maintainability,
and enable static analysis.
Changed import paths from google.ads.googleads.v17 to google.ads.googleads.v20 and updated GoogleAdsClient.load_from_storage to use version='v20'.
@BenRKarl BenRKarl force-pushed the add-type-hints-shopping-ads branch from 5fc7b4b to 2eb243a Compare August 1, 2025 15:39
Change-Id: I836105786063f23d40941a8f8f1986d3d48e2253
@sundquist sundquist self-requested a review August 1, 2025 19:33
Copy link
Contributor

@sundquist sundquist left a comment

Choose a reason for hiding this comment

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

LGTM

@BenRKarl BenRKarl merged commit bc9724c into main Aug 1, 2025
3 checks passed
@BenRKarl BenRKarl deleted the add-type-hints-shopping-ads branch August 1, 2025 19:37
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.

4 participants