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

feat(genpapic): support protobuf-go go_package mapping option #1029

Merged
merged 1 commit into from
Jun 10, 2022

Conversation

noahdietz
Copy link
Collaborator

Per the protobuf Go docs, there is an option to override the go_package option in specific proto files. If this is used during generation of the protobuf/grpc stubs, the GAPIC generator needs to know about it, so that the proper imports can be generated for the request/response/stub types.

@noahdietz noahdietz requested review from a team as code owners June 10, 2022 18:10
Copy link
Member

@codyoss codyoss left a comment

Choose a reason for hiding this comment

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

LGTM!

@noahdietz noahdietz merged commit f40c830 into googleapis:main Jun 10, 2022
@noahdietz noahdietz deleted the remap-proto-pkg branch June 10, 2022 18:17
@noahdietz noahdietz restored the remap-proto-pkg branch June 10, 2022 18:17
@noahdietz noahdietz deleted the remap-proto-pkg branch June 10, 2022 18:18
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.

3 participants