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

Pub/Sub autogen refresh #4297

Closed

Conversation

lukesneeringer
Copy link
Contributor

This is a merger of Pub/Sub with google.api_core. Marked do not merge while Ernest and I work out some khinks.

@lukesneeringer lukesneeringer added api: pubsub Issues related to the Pub/Sub API. do not merge Indicates a pull request not ready for merge, due to either quality or timing. labels Oct 31, 2017
@lukesneeringer lukesneeringer self-assigned this Oct 31, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 31, 2017
@lukesneeringer lukesneeringer removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Oct 31, 2017
@lukesneeringer
Copy link
Contributor Author

This now seems to be in good shape. Lobbing to @jonparrott for review.

@dhermes
Copy link
Contributor

dhermes commented Oct 31, 2017

@lukesneeringer FYI #4296 is an uber-release, but I was planning on checking on you before actually pushing the tag for Pub/Sub.

Copy link
Contributor

@theacodes theacodes left a comment

Choose a reason for hiding this comment

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

Mostly LGTM

from google.protobuf import field_mask_pb2

_GAPIC_LIBRARY_VERSION = pkg_resources.get_distribution(
'google-cloud-pubsub', ).version

This comment was marked as spam.

This comment was marked as spam.

credentials=credentials,
scopes=self._DEFAULT_SCOPES)

self.iam_policy_stub = (iam_policy_pb2.IAMPolicyStub(channel))

This comment was marked as spam.

@tseaver tseaver added priority: p2 Moderately-important priority. Fix may not be included in next release. and removed priority: p2 Moderately-important priority. Fix may not be included in next release. labels Nov 6, 2017
@dhermes dhermes mentioned this pull request Nov 22, 2017
@dhermes
Copy link
Contributor

dhermes commented Nov 22, 2017

Closing in favor or #4438 (which literally pushed this exact set of history to start). But that PR is in the GoogleCloudPlatform remote so we can run system tests.

@dhermes dhermes closed this Nov 22, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants