-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Removing _gcloud_vendor. #808
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
Conversation
|
so there were two separate issues here:
|
|
|
|
DERP. Thanks @craigcitro! I'm going to close this out and send a new PR. Note that we can't "accept" the PR unless google/apitools@a6feb8a is into a PyPI release. |
|
done -- v0.4.2 is on pypi. |
* fix: Allow protobuf 5.x; require protobuf >=3.20.2 * update constraints --------- Co-authored-by: Chalmer Lowe <[email protected]>
* fix: Allow protobuf 5.x; require protobuf >=3.20.2 * update constraints --------- Co-authored-by: Chalmer Lowe <[email protected]>
Source-Link: googleapis/synthtool@e6f91eb Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-python:latest@sha256:365d92ef2206cfad00a8c5955c36789d0de124e2b6d92a72dd0486315a0f2e57 Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
* refactor: remove duplication in grpc transport init * refactor: let base transport set final host, creds, scopes * refactor: remove scopes or self.AUTH_SCOPES in grpc * docs: explain self._prep_wrapped_messages
…talled (#808) Co-authored-by: ohmayr <[email protected]>
* docs: remove line about Python 2.7 being unsupported According to the README and setup.py, we do support Python 2.7. Also clarify that the utils module and its positional decorator do still exist and does work in both Py2 and Py3 * Update migrating.rst
@craigcitro Removing our vendored in code and switching to the PyPI version of
apitoolsgives 4 errors I can't readily grok. (This was similarly reported by @tseaver.)Care to take a look and check out my branch?