-
Notifications
You must be signed in to change notification settings - Fork 58
Switch to rust-vmm-ci #14
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
Conversation
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
cacae16
to
698de33
Compare
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
@andreeaflorescu the build is successful, but the webhook didn't get the message, help? 😃 |
This was some draft PR black magic. Once updated to ready-for-review PR, the correct status is reported. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The tests directory needs to be removed as well as we are using the coverage test from rust-vmm-ci.
@@ -18,16 +18,17 @@ cargo build | |||
Our Continuous Integration (CI) pipeline is implemented on top of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, the readme should be updated as well now that we use the rust-vmm-ci directly. In other repositories we just linked the rust-vmm-ci and got rid of all the other details.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Done ✔️
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
Signed-off-by: Alexandra Iordache <aghecen@amazon.com>
Done |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
rslgtm
Fixes #4
rust-vmm-ci
pipeline, using the one from the submodule instead(Still not very sure of what I'm doing, please call out any stupid stuff)