Skip to content

feat(networkconnectivity): initial generation of apiv1alpha1#3567

Merged
gcf-merge-on-green[bot] merged 7 commits intomasterfrom
networkconnectivity
Jan 25, 2021
Merged

feat(networkconnectivity): initial generation of apiv1alpha1#3567
gcf-merge-on-green[bot] merged 7 commits intomasterfrom
networkconnectivity

Conversation

@bcoe
Copy link
Copy Markdown

@bcoe bcoe commented Jan 15, 2021

Initial generation of network connectivity.

@bcoe bcoe requested a review from a team January 15, 2021 02:30
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 15, 2021
@bcoe bcoe requested a review from codyoss January 15, 2021 02:32
Comment thread internal/gapicgen/generator/config.go Outdated
Benjamin E. Coe and others added 2 commits January 15, 2021 14:58
Co-authored-by: Cody Oss <6331106+codyoss@users.noreply.github.com>
@bcoe bcoe requested a review from codyoss January 16, 2021 01:13
Copy link
Copy Markdown
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.

It looks like genlocal still need to be run again. Doing so will move the generated artifacts to a new folder. You might need to manually delete out the file that were generated in the wrong folder.

@codyoss codyoss changed the title feat(networkconnectivity): initial generation of v1alpha1 feat(networkconnectivity): initial generation of apiv1alpha1 Jan 19, 2021
@bcoe bcoe requested a review from codyoss January 25, 2021 17:36
Copy link
Copy Markdown
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, thanks!

@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Jan 25, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit adf489a into master Jan 25, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the networkconnectivity branch January 25, 2021 19:28
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 25, 2021
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