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

Re-generate library using pubsub/synth.py by dpebot · Pull Request #6223 · googleapis/google-cloud-python · GitHub

Re-generate library using pubsub/synth.py - #6223

Merged
tseaver merged 1 commit into
masterfrom
autosynth-pubsub
Oct 17, 2018
Merged

Re-generate library using pubsub/synth.py#6223
tseaver merged 1 commit into
masterfrom
autosynth-pubsub

Conversation

dpebot commented Oct 16, 2018

Copy link
Copy Markdown
Contributor

This PR was created by autosynth.

dpebot requested a review from crwilcox as a code owner October 16, 2018 16:37
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 16, 2018
tseaver added codegen api: pubsub Issues related to the Pub/Sub API. labels Oct 17, 2018

tseaver commented Oct 17, 2018

Copy link
Copy Markdown
Contributor

Pubsub tests did not actually run, due to #6231. Please hold off merging until that issue is fixed, and then force a rebuild for the Kokoro Pubsub job(how?)

tseaver added do not merge Indicates a pull request not ready for merge, due to either quality or timing. status: blocked Resolving the issue is dependent on other work. labels Oct 17, 2018
crwilcox added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 17, 2018
kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 17, 2018
tseaver added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 17, 2018
kokoro-team removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Oct 17, 2018

tseaver commented Oct 17, 2018

Copy link
Copy Markdown
Contributor

Attempting to force Kokoro to run: I don't know if it runs against a trial merge of current master, or the PRs own branch, so fixes in #6242 and #6243 may not help.

tseaver removed do not merge Indicates a pull request not ready for merge, due to either quality or timing. status: blocked Resolving the issue is dependent on other work. labels Oct 17, 2018

tseaver commented Oct 17, 2018

Copy link
Copy Markdown
Contributor

Storage test failure (#62244) is unrelated.

tseaver merged commit c8d4f5d into master Oct 17, 2018
tseaver deleted the autosynth-pubsub branch October 17, 2018 20:41
parthea pushed a commit that referenced this pull request Mar 2, 2026
Re-generate library using 'pubsub/synth.py'.
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: pubsub Issues related to the Pub/Sub API. cla: yes This human has signed the Contributor License Agreement. codegen

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL