Skip to content

Video Intelligence: Remove previous auto-gen.#3974

Closed
lukesneeringer wants to merge 2 commits into
googleapis:masterfrom
lukesneeringer:vi-remove-autogen
Closed

Video Intelligence: Remove previous auto-gen.#3974
lukesneeringer wants to merge 2 commits into
googleapis:masterfrom
lukesneeringer:vi-remove-autogen

Conversation

@lukesneeringer

Copy link
Copy Markdown
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

@dhermes dhermes 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.

Rubber stamp LGTM

@dhermes

dhermes commented Sep 15, 2017

Copy link
Copy Markdown
Contributor

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

lukesneeringer commented Sep 15, 2017

Copy link
Copy Markdown
Contributor Author

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