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

Combine with vm-operator-api #9

Open
janitha opened this issue Apr 1, 2021 · 2 comments
Open

Combine with vm-operator-api #9

janitha opened this issue Apr 1, 2021 · 2 comments

Comments

@janitha
Copy link
Contributor

janitha commented Apr 1, 2021

Currently the vm-operator's api resides in https://github.com/vmware-tanzu/vm-operator-api this should be brought in to this repo.

@dramdass
Copy link
Contributor

dramdass commented Apr 1, 2021

I think it still warrants some discussion whether this should be combined. Clients who vendor the API may not want to bring in all the dependencies brought in by the implementation.

@timothysc
Copy link

It's common practice to contain everything in the main repo primarily for re-visioning. So long as you manage your type deps there is no reason not to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants
@janitha @timothysc @dramdass and others