FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Use googleapis common protos by dhermes · Pull Request #1353 · googleapis/google-cloud-python · GitHub

Use googleapis common protos - #1353

Merged
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:use-googleapis-common-protos
Jan 15, 2016
Merged

Use googleapis common protos#1353
dhermes merged 1 commit into
googleapis:masterfrom
dhermes:use-googleapis-common-protos

Conversation

dhermes commented Jan 7, 2016

Copy link
Copy Markdown
Contributor

Has #1317 as diffbase.

H/T to @tbetbetbe who alerted me to the existence of https://pypi.python.org/pypi/googleapis-common-protos. w00t!

dhermes added packaging api: bigtable Issues related to the Bigtable API. labels Jan 7, 2016
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jan 7, 2016
dhermes force-pushed the use-googleapis-common-protos branch from 8edeeb9 to 621e354 Compare January 7, 2016 23:46
tseaver referenced this pull request in dhermes/google-cloud-python Jan 14, 2016
This is in advance of `v1beta3`, where it will be needed
to parse `Value.timestamp_value` (which is of type
`timestamp_pb2.Timestamp`). Also adding `_datetime_to_pb_timestamp`
for the other direction.
dhermes force-pushed the use-googleapis-common-protos branch from 621e354 to 02553e6 Compare January 14, 2016 21:32
dhermes force-pushed the use-googleapis-common-protos branch from 02553e6 to 4def774 Compare January 14, 2016 23:19

dhermes commented Jan 14, 2016

Copy link
Copy Markdown
Contributor Author

@tseaver PTAL I rebased after #1317

Comment thread scripts/make_operations_grpc.py Outdated

This comment was marked as spam.

This comment was marked as spam.

tseaver commented Jan 15, 2016

Copy link
Copy Markdown
Contributor

LGTM, even if you decide not to address the earlier writelines comment.

dhermes force-pushed the use-googleapis-common-protos branch from 4def774 to 3a58fee Compare January 15, 2016 02:19
dhermes added a commit that referenced this pull request Jan 15, 2016
dhermes merged commit e1bc703 into googleapis:master Jan 15, 2016
dhermes deleted the use-googleapis-common-protos branch January 15, 2016 02:25
dhermes added a commit to dhermes/google-cloud-python that referenced this pull request Jan 15, 2016
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
parthea pushed a commit that referenced this pull request Mar 9, 2026
#1353)

* test: test universe domain client only in prod

* unflake hmac snippet test
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: bigtable Issues related to the Bigtable API. cla: yes This human has signed the Contributor License Agreement. packaging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants


Back | FazBrowse Home | New Git URL