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.

Sign up for GitHub

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

Jump to bottom

Video Intelligence (GAPIC only) #2318

Merged
merged 14 commits into from
May 18, 2017
Merged

Video Intelligence (GAPIC only) #2318

merged 14 commits into from
May 18, 2017

Conversation

lukesneeringer
Copy link
Contributor

This is the video intelligence (GAPIC only) release for Node.js.

@lukesneeringer lukesneeringer self-assigned this May 17, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label May 17, 2017
@lukesneeringer lukesneeringer requested a review from jmdobry May 17, 2017 19:23
stephenplusplus
stephenplusplus reviewed May 17, 2017
View reviewed changes
packages/video-intelligence/package.json Outdated
{
"url": "https://github.com/googleapis/googleapis",
"name": "@google-cloud/video-intelligence",
"version": "0.1.0",

This comment was marked as spam.

Sign in to view

packages/video-intelligence/package.json Outdated
"node": ">=4.0.0"
},
"scripts": {
"test": "mocha"

This comment was marked as spam.

Sign in to view

packages/video-intelligence/src/index.js Outdated
@@ -0,0 +1,55 @@
/*
* Copyright 2016 Google Inc. All rights reserved.

This comment was marked as spam.

Sign in to view

packages/video-intelligence/src/index.js Outdated
'use strict';

var gapic = {
v1beta1: require('./v1beta1'),

This comment was marked as spam.

Sign in to view

packages/video-intelligence/src/index.js Outdated


/**
* Create an imageAnnotatorClient with additional helpers for common

This comment was marked as spam.

Sign in to view

packages/video-intelligence/src/index.js Outdated
* Create an imageAnnotatorClient with additional helpers for common
* tasks.
*
* @param {Object=} opts - The optional parameters.

This comment was marked as spam.

Sign in to view

stephenplusplus
stephenplusplus reviewed May 17, 2017
View reviewed changes
packages/video-intelligence/package.json Outdated
@@ -0,0 +1,27 @@
{
"url": "https://github.com/googleapis/googleapis",

This comment was marked as spam.

Sign in to view

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.925% when pulling df81904 on videointelligence into 08cdfe0 on master.

@googlebot
Copy link

We found a Contributor License Agreement for you (the sender of this pull request), but were unable to find agreements for the commit author(s). If you authored these, maybe you used a different email address in the git commits than was used to sign the CLA (login here to double check)? If these were authored by someone else, then they will need to sign a CLA as well, and confirm that they're okay with these being contributed to Google.

@googlebot googlebot added cla: no This human has *not* signed the Contributor License Agreement. and removed cla: yes This human has signed the Contributor License Agreement. labels May 18, 2017
stephenplusplus
stephenplusplus reviewed May 18, 2017
View reviewed changes
packages/video-intelligence/src/index.js Outdated
module.exports = videointelligenceV1beta1;
module.exports.v1beta1 = videointelligenceV1beta1;
module.exports = VideoIntelligence;
module.exports.v1beta1 = v1beta1;

This comment was marked as spam.

Sign in to view

stephenplusplus
stephenplusplus reviewed May 18, 2017
View reviewed changes
packages/video-intelligence/src/index.js Outdated
* The domain name of the API remote host.
* @param {number=} opts.port
* The port on which to connect to the remote host.
* @param {grpc.ClientCredentials=} opts.sslCreds

This comment was marked as spam.

Sign in to view

@stephenplusplus
Copy link
Contributor

stephenplusplus commented May 18, 2017 •
edited
Loading

@lukesneeringer PTAL and feel free to merge. That will be my queue cue (too much async) to get out the 0.1.0 release.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 99.925% when pulling bb3c63f on videointelligence into 08cdfe0 on master.

@stephenplusplus stephenplusplus merged commit 52fdac9 into master May 18, 2017
@stephenplusplus stephenplusplus deleted the videointelligence branch May 18, 2017 01:06
@stephenplusplus stephenplusplus added cla: yes This human has signed the Contributor License Agreement. and removed cla: no This human has *not* signed the Contributor License Agreement. labels May 18, 2017
This was referenced Jul 5, 2017
Merged
Merged
sofisl pushed a commit that referenced this pull request Sep 27, 2022
@lukesneeringer @stephenplusplus
Video Intelligence (GAPIC only) (#2318)
78f253c
@sofisl sofisl mentioned this pull request Sep 27, 2022
Closed
4 tasks
This was referenced Nov 9, 2022
Closed
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@stephenplusplus stephenplusplus stephenplusplus left review comments

@jmdobry jmdobry Awaiting requested review from jmdobry

Assignees

@lukesneeringer lukesneeringer

Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lukesneeringer @coveralls @googlebot @stephenplusplus