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.

Sign up for GitHub

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

Jump to bottom

prepare 0.8.0 release #193

Merged
merged 1 commit into from
Apr 7, 2022
Merged

prepare 0.8.0 release #193

merged 1 commit into from
Apr 7, 2022

Conversation

lauralt
Copy link
Contributor

@lauralt lauralt commented Mar 17, 2022 •
edited by andreeaflorescu
Loading

Signed-off-by: Laura Loghin [email protected]

Summary of the PR

Let's have a release with #190 as it is actually a pretty ugly bug.

Requirements

Before submitting your PR, please make sure you addressed the following
requirements:

  • All commits in this PR are signed (with git commit -s), and the commit
    message has max 60 characters for the summary and max 75 characters for each
    description line.
  • All added/changed functionality has a corresponding unit/integration
    test.
  • Any newly added unsafe code is properly documented.

@lauralt
prepare 0.8.0 release
764d88d 
Signed-off-by: Laura Loghin <[email protected]>
@jiangliu
Copy link
Member

It would be great to also include #192

@lauralt
Copy link
Contributor Author

lauralt commented Mar 17, 2022

It would be great to also include #192

@alexandruag had some concerns around it. Let's see what he thinks about it, and then we can proceed with the release as well.

@lauralt
Copy link
Contributor Author

lauralt commented Apr 4, 2022 •
edited
Loading

@jiangliu Can we proceed with this one? Looks like it may take some more time to have a decision on #192.

@andreeaflorescu
Copy link
Member

Let's merge this one, and we can do another release after #192 is also merged.

@lauralt lauralt merged commit fc7153a into rust-vmm:main Apr 7, 2022
@lauralt lauralt deleted the release-0.8.0 branch April 7, 2022 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jiangliu jiangliu jiangliu approved these changes

@andreeaflorescu andreeaflorescu andreeaflorescu approved these changes

@alexandruag alexandruag Awaiting requested review from alexandruag alexandruag is a code owner

@bonzini bonzini Awaiting requested review from bonzini bonzini is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@lauralt @jiangliu @andreeaflorescu