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

Move UI plugin registration codes to vic-product #8213

Closed
wants to merge 3 commits into from

Conversation

wjun
Copy link
Contributor

@wjun wjun commented Aug 16, 2018

Jason Morris and others added 3 commits May 18, 2018 16:00
Removes vic ui code and ci rules from
this this repo for inclusion in vic-product.
Fixes vmware/vic-product#1433.
Moves lib/install/ova/ and lib/install/plugin to vic-product for
use by the fileserver plugin install. Fixes vmware/vic-product#1702
@wjun wjun requested a review from a team as a code owner August 16, 2018 06:50
@wjun wjun requested review from a user and zjs August 16, 2018 06:53
@zjs
Copy link
Member

zjs commented Aug 16, 2018

Does this PR take into consideration the notes in vmware/vic-product#1432 (comment)?

@zjs
Copy link
Member

zjs commented Aug 16, 2018

Fixes vmware/vic-product/#1432

I think this PR makes progress towards this issue, but does not fix it.

Copy link
Member

@zjs zjs left a comment

Choose a reason for hiding this comment

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

At the very least, this should not be merged to master with a merge commit from master.

@wjun
Copy link
Contributor Author

wjun commented Aug 17, 2018

@zjs Thanks for the reminder. I have changed the the feature branch and rebased &resolved the conflicts at #8219 for review. I will abandon this pr.

@wjun wjun closed this Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants