Skip to content

Correct speech Operation docstring - #3379

Merged
dhermes merged 1 commit into
googleapis:masterfrom
cfperez:cfperez-operation
May 5, 2017
Merged

Correct speech Operation docstring#3379
dhermes merged 1 commit into
googleapis:masterfrom
cfperez:cfperez-operation

Conversation

@cfperez

Copy link
Copy Markdown
Contributor

ValueError is raised when there are no results, not when there is more than one result, which is what the docstring says.

ValueError is raised when there are no results, not when there is more than one result, which is what the docstring says.
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label May 5, 2017

@cfperezcfperez left a comment

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See L64

@dhermes

Copy link
Copy Markdown
Contributor

LGTM

@dhermes

Copy link
Copy Markdown
Contributor

Thanks @cfperez. No need to wait on CI since your change is docs only (and even more, these docs don't get built since a non-public method)

@dhermes
dhermes merged commit 98c9cb7 into googleapis:masterMay 5, 2017
@tseavertseaver mentioned this pull request Jun 26, 2017
atulep pushed a commit that referenced this pull request Apr 3, 2023
ValueError is raised when there are no results, not when there is more than one result, which is what the docstring says.
atulep pushed a commit that referenced this pull request Apr 18, 2023
ValueError is raised when there are no results, not when there is more than one result, which is what the docstring says.
parthea pushed a commit that referenced this pull request Oct 22, 2023
ValueError is raised when there are no results, not when there is more than one result, which is what the docstring says.
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@cfperez@dhermes@googlebot