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

chore: add preserve_gapic to java library generation options #1160

Merged
merged 4 commits into from
Jul 20, 2021
Merged

chore: add preserve_gapic to java library generation options #1160

merged 4 commits into from
Jul 20, 2021

Conversation

BenWhitehead
Copy link
Contributor

There are cases when generating a library the gapic directory prefix should be preserved (veneer clients for example). This feature adds a new parameter preserve_gapic to java.gapic_library and java.bazel_library

There are cases when generating a library, the gapic directory prefix should be preserved (veneer clients for example). This feature adds a new parameter `preserve_gapic` to `java.gapic_library` and `java.bazel_library`.
@BenWhitehead BenWhitehead requested a review from a team July 16, 2021 21:50
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 16, 2021
@chingor13 chingor13 changed the title feat: add preserve_gapic to java library generation options chore: add preserve_gapic to java library generation options Jul 20, 2021
@chingor13
Copy link
Contributor

Changed to a "chore" because this does trigger a new docker image build and the commit message would be a feat.

@chingor13 chingor13 added the automerge Merge the pull request once unit tests and other checks pass. label Jul 20, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit f330962 into googleapis:master Jul 20, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jul 20, 2021
@BenWhitehead BenWhitehead deleted the preserve_gapic branch July 20, 2021 21:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants