Skip to content

Conversation

@msdinit
Copy link
Contributor

@msdinit msdinit commented Dec 28, 2018

This example is consistent with a similar change implemented in nodejs version here: googleapis/nodejs-tasks#142

@msdinit msdinit requested a review from crwilcox as a code owner December 28, 2018 15:29
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Dec 28, 2018
@tseaver tseaver changed the title Add working example for cloud tasks create_queue Tasks: Add working example for 'create_queue' Jan 3, 2019
@tseaver tseaver added type: docs Improvement to the documentation for an API. api: cloudtasks Issues related to the Cloud Tasks API. labels Jan 3, 2019
@tseaver tseaver merged commit 2a1341c into googleapis:master Jan 3, 2019
@tseaver
Copy link
Contributor

tseaver commented Jan 5, 2019

@msdinit Thanks for the patch! Unfortunately, that docstring is in a module which is generated from the comments in the relevant .proto file, and will get overwritten by subsequent auto-generation runs (see #7057).

@tseaver
Copy link
Contributor

tseaver commented Jan 5, 2019

Also, now that I look at it, the doctest example is actually invalid: it needs ... as the prefix for the lines inside the opened {.

tseaver added a commit that referenced this pull request Jan 5, 2019
Also, remove now-spurious replacements.

Supersedes #7057.
parthea pushed a commit that referenced this pull request Sep 22, 2023
Also, remove now-spurious replacements.

Supersedes #7057.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: cloudtasks Issues related to the Cloud Tasks API. cla: yes This human has signed the Contributor License Agreement. type: docs Improvement to the documentation for an API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants