Skip to content

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