Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(gengapic): document client/bidi streaming unsupported in REST #1181

Merged
merged 1 commit into from Nov 8, 2022

Conversation

noahdietz
Copy link
Collaborator

Generate interface-level, method documentation for Client and Bi-Di streaming methods when rest transport is selected that indicates that the method is unsupported for the REST transport. This should help reduce confusion when the method returns an error saying the same thing.

Note: m.GetClientStreaming() covers both streaming types in scope becaus Bi-Di stream is just ClientStreaming: true && ServerStreaming: true.

@noahdietz noahdietz requested review from a team as code owners November 8, 2022 22:29
@noahdietz noahdietz merged commit f9a9191 into googleapis:main Nov 8, 2022
gcf-merge-on-green bot pushed a commit that referenced this pull request Nov 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants