Skip to content

Conversation

@tseaver
Copy link
Contributor

@tseaver tseaver commented Mar 26, 2015

No description provided.

@tseaver tseaver added the api: pubsub Issues related to the Pub/Sub API. label Mar 26, 2015
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 26, 2015
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 96b5925 on tseaver:pubsub-implicit_environ into d34a15e on GoogleCloudPlatform:master.

@tseaver tseaver mentioned this pull request Mar 27, 2015
9 tasks
@dhermes
Copy link
Contributor

dhermes commented Mar 27, 2015

Just reviewed from phone. LGTM.

I'm unclear if we need both the Cloud Platform scope and the Pubsub scope (i.e. just one will do).

Maybe not though. There are methods in storage where the storage.full_control is the only scope that will work.

tseaver added a commit that referenced this pull request Mar 27, 2015
Add implicit environment support to pubsub
@tseaver tseaver merged commit b45a909 into googleapis:master Mar 27, 2015
@tseaver tseaver deleted the pubsub-implicit_environ branch March 27, 2015 16:18
@tseaver
Copy link
Contributor Author

tseaver commented Mar 27, 2015

@dhermes the API docs for authorization appear to indicate that either scope would be fine.

@tmatsuo
Copy link
Contributor

tmatsuo commented Mar 27, 2015

In my understanding, the Cloud Platform scope contains all of the scopes. I prefer using only Pubsub scope explicitly rather than using the wildcard.

parthea pushed a commit that referenced this pull request Aug 21, 2025
* chore: exclude sample-specific formatting from owlbot

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Sep 16, 2025
* chore: exclude sample-specific formatting from owlbot

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
parthea pushed a commit that referenced this pull request Sep 18, 2025
* specify a particular version of bigquery to debug

* again tweaking the versions to debug issue.

* add some pip freeze commands for debugging

* updates minimum latency to correct a flaky bot issue and protect users

* Update noxfile.py

* Update noxfile.py

* Update setup.py

* Update noxfile.py

* add several test cases to test validation logic

* 🦉 Updates from OwlBot post-processor

See https://github.com/googleapis/repo-automation-bots/blob/main/packages/owl-bot/README.md

---------

Co-authored-by: Owl Bot <gcf-owl-bot[bot]@users.noreply.github.com>
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.

5 participants