Skip to content

chore: add preserve_gapic to java library generation options#1160

Merged
gcf-merge-on-green[bot] merged 4 commits intogoogleapis:masterfrom
BenWhitehead:preserve_gapic
Jul 20, 2021
Merged

chore: add preserve_gapic to java library generation options#1160
gcf-merge-on-green[bot] merged 4 commits intogoogleapis:masterfrom
BenWhitehead:preserve_gapic

Conversation

@BenWhitehead
Copy link
Copy Markdown
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
Copy Markdown
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