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: start generating storage v2 gapic #922

Merged
merged 4 commits into from
Aug 11, 2021

Conversation

noahdietz
Copy link
Contributor

Adds GAPIC generation target for Storage API v2.

@noahdietz noahdietz requested a review from a team July 14, 2021 22:45
@noahdietz noahdietz requested a review from a team as a code owner July 14, 2021 22:45
@product-auto-label product-auto-label bot added the api: storage Issues related to the googleapis/java-storage API. label Jul 14, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jul 14, 2021
@noahdietz
Copy link
Contributor Author

@BenWhitehead @danielduhh beginning generation of the Java GAPIC for v2 was requested by the API team. I've never set up a Java GAPIC in a repo before so I'm not sure what should be done. Ben, do you know how to wire things up dependency wise?

We probably want to add some disclaimer that this client shouldn't be used directly and that the Veneer should be used still.

@BenWhitehead
Copy link
Collaborator

We'll need googleapis/synthtool#1160 to merge into synthool before we can merge this.

@BenWhitehead BenWhitehead added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jul 16, 2021
@BenWhitehead BenWhitehead self-assigned this Jul 16, 2021
@BenWhitehead BenWhitehead force-pushed the v2-gapic branch 2 times, most recently from 5cfab7f to d6acdab Compare July 16, 2021 23:25
@BenWhitehead
Copy link
Collaborator

Issue for name collisions with Object hopefully will be fixed in the next few weeks. googleapis/sdk-platform-java#798

@BenWhitehead BenWhitehead force-pushed the v2-gapic branch 2 times, most recently from 730b270 to cb78bfc Compare August 11, 2021 20:38
@BenWhitehead BenWhitehead removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Aug 11, 2021
@BenWhitehead
Copy link
Collaborator

Failing dependency builds will pass after code is generated, I'm going to manually trigger it now.

@BenWhitehead BenWhitehead merged commit 0ea286b into googleapis:master Aug 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: storage Issues related to the googleapis/java-storage API. 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