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

Add initial readme, license and CI #1

Merged
merged 1 commit into from
Sep 22, 2020
Merged

Conversation

alxiord
Copy link
Member

@alxiord alxiord commented Aug 24, 2020

No description provided.

Copy link
Member

@andreeaflorescu andreeaflorescu left a comment

Choose a reason for hiding this comment

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

High level it looks good to me. I think it would be interesting to add more details about the design (i.e. interfaces that we expect to have, high level components?). Are we going to add these details in a subsequent PR?

@alxiord
Copy link
Member Author

alxiord commented Aug 24, 2020

High level it looks good to me. I think it would be interesting to add more details about the design (i.e. interfaces that we expect to have, high level components?). Are we going to add these details in a subsequent PR?

Yup. I'd like to have the skeleton in place, and follow up with details (as they become apparent to me). It'd also be great to get a discussion going here on what folks expect from the reference VMM.

@alxiord
Copy link
Member Author

alxiord commented Aug 26, 2020

@michael2012z WDYT about the (very high level) proposal in the readme?

@michael2012z
Copy link

@aghecenco It's a good high level description. The purposes of the reference VMM are well defined.

Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
@lauralt lauralt mentioned this pull request Sep 8, 2020
@alxiord alxiord merged commit 8d343a5 into rust-vmm:master Sep 22, 2020
@alxiord alxiord deleted the init branch October 27, 2020 08:19
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

Successfully merging this pull request may close these issues.

5 participants