Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Add GAX-based _SubscriberAPI.#1795

Merged
tseaver merged 3 commits into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
tseaver:pubsub-wrap_gax_subscriber_apiCopy head branch name to clipboard
May 16, 2016
Merged

Add GAX-based _SubscriberAPI.#1795
tseaver merged 3 commits into
googleapis:mastergoogleapis/google-cloud-python:masterfrom
tseaver:pubsub-wrap_gax_subscriber_apiCopy head branch name to clipboard

Conversation

@tseaver

@tseaver tseaver commented May 13, 2016

Copy link
Copy Markdown
Contributor

Uses #1764 as a base.

@tseaver tseaver added the api: pubsub Issues related to the Pub/Sub API. label May 13, 2016
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 13, 2016
@tseaver

tseaver commented May 16, 2016

Copy link
Copy Markdown
Contributor Author

@dhermes rebased after merging #1794 to master. PTAL

Comment thread gcloud/pubsub/_gax.py
"""List subscriptions for the project associated with this API.

See:
https://cloud.google.com/pubsub/reference/rest/v1/projects.subscriptions/list

This comment was marked as spam.

This comment was marked as spam.

@dhermes

dhermes commented May 16, 2016

Copy link
Copy Markdown
Contributor

LGTM. Please fix the small nits, but don't need to check with me once fixed.

@tseaver tseaver merged commit bb26745 into googleapis:master May 16, 2016
@tseaver tseaver deleted the pubsub-wrap_gax_subscriber_api branch May 16, 2016 15:54
@tseaver tseaver mentioned this pull request May 16, 2016
parthea pushed a commit that referenced this pull request Nov 26, 2025
* update secrets

* 1

* 2
parthea pushed a commit that referenced this pull request Apr 1, 2026
* test: avoid exact float comparison in `test_apply_lambda`

* use by_row=False in apply_simple_udf too
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.

3 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.