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

Video Intelligence: Remove previous auto-gen. #3974

Closed

Conversation

lukesneeringer
Copy link
Contributor

This removes the previous auto-gen surface from video intelligence. The tests will fail until a follow-up PR that is based on this one is merged in.

Equivalent of #3963, but for video intelligence.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Sep 15, 2017
Copy link
Contributor

@dhermes dhermes left a comment

Choose a reason for hiding this comment

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

Rubber stamp LGTM

@dhermes
Copy link
Contributor

dhermes commented Sep 15, 2017

ASIDE: @lukesneeringer Could you have more descriptive PR names? Here really would just be nice to say which package is affected. (The end of the monorepo would make it moot.)

@lukesneeringer lukesneeringer changed the title Remove previous auto-gen. Video Intelligence: Remove previous auto-gen. Sep 15, 2017
@lukesneeringer
Copy link
Contributor Author

lukesneeringer commented Sep 15, 2017

Re-opening a new PR not on my fork.

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.

3 participants