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

Q1 2024 Release #830

Merged
merged 8 commits into from
Jan 23, 2024
Merged

Q1 2024 Release #830

merged 8 commits into from
Jan 23, 2024

Conversation

BenRKarl
Copy link
Contributor

@BenRKarl BenRKarl commented Jan 12, 2024

  • Remove support for Python 3.7
  • Add support for Python 3.11 and 3.12
  • Remove support for Google Ads API v13
  • Add Docker support for unit tests
  • Switch to a native/implicit namespace package
  • Remove usage of pkg_resources and distutils in client code and generated code.

Remove support for Python 3.7
Add Python 3.8, 3.9 and 3.10 to unit tests runs.

Instructions to run unit tests:

1. Without Docker:
  * Make sure you have pyenv installed (go/adr-python).
  * Make sure Python versions 3.8, 3.9 and 3.10 are installed.
  * From project root run: pyenv local 3.8 3.9 3.10
  * From project root run: nox
2. With Docker:
  * Make sure you have Docker installed.
  * From project root run: docker build -t google-ads-python:test .
  * From project root run: docker run -it google-ads-python:test cd nox

Change-Id: I64b1fa622143dced50c690f69410e84f7b7d27bf
Python 3.11 and 3.12.

Change-Id: Ie33c74a1f74418550d49e74670c6328eea4279d8
  * Removes google/__init__.py and google/ads/__init__.py
  * Switch to find_namespace_packages in setup.py
2. Remove usage of pkg_resources
  * Regenerate v13, v14, and v15 stubs with gapic generator v1.13.0
  * Remove from googleads/client.py and rely on default transport
    parameter
  * Remove from metadata interceptor and reply on __version__ for pb
    version.

Change-Id: I7fec326731ac321be131270843aba6cf5d76cff8
@BenRKarl BenRKarl requested a review from a team as a code owner January 12, 2024 19:14
@BenRKarl BenRKarl requested review from bobhancockg and AnashOommen and removed request for bobhancockg January 12, 2024 19:14
AnashOommen
AnashOommen previously approved these changes Jan 12, 2024
google/ads/googleads/config.py Outdated Show resolved Hide resolved
google/ads/googleads/config.py Outdated Show resolved Hide resolved
google/ads/googleads/v13/__init__.py Outdated Show resolved Hide resolved
Dockerfile Outdated Show resolved Hide resolved
bobhancockg
bobhancockg previously approved these changes Jan 17, 2024
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