Skip to content

Add vision GAPIC - #463

Merged
jdpedrie merged 3 commits into
googleapis:masterfrom
michaelbausor:add-vision-gapic
Apr 25, 2017
Merged

Add vision GAPIC#463
jdpedrie merged 3 commits into
googleapis:masterfrom
michaelbausor:add-vision-gapic

Conversation

@michaelbausor

Copy link
Copy Markdown
Contributor

Support for doc generation coming

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Apr 24, 2017
@michaelbausor michaelbausor changed the title WIP: Add vision GAPIC Add vision GAPIC Apr 25, 2017

@dwsupplee dwsupplee left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM. Is there anything else we need or can this be merged and set up for a release?

Comment thread docs/contents/cloud-vision.json Outdated
],
"nav": [{
"title": "VisionClient",
"type": "vision/v1/visionclient"

This comment was marked as spam.

This comment was marked as spam.

Comment thread docs/contents/cloud-vision.json Outdated
"vision/v1/\\w{1,}"
],
"nav": [{
"title": "VisionClient",

This comment was marked as spam.

@jdpedrie

Copy link
Copy Markdown
Contributor

Is the TOC incorrect, or the gapic client name?

@michaelbausor

Copy link
Copy Markdown
Contributor Author

@jdpedrie Good spot, the TOC was incorrect, updated.

@dwsupplee I don't think we need anything else.

@jdpedrie

Copy link
Copy Markdown
Contributor

LGTM!

@jdpedrie
jdpedrie merged commit f684a02 into googleapis:master Apr 25, 2017
@dwsupplee dwsupplee mentioned this pull request Apr 25, 2017
@michaelbausor
michaelbausor deleted the add-vision-gapic branch May 12, 2017 18:13
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.

4 participants