-
Notifications
You must be signed in to change notification settings - Fork 7
Add CI #3
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 CI #3
Conversation
Taken from runc. Signed-off-by: Kir Kolyshkin <[email protected]>
Taken from runc. Signed-off-by: Kir Kolyshkin <[email protected]>
Also, fix a typo found in RELEASES.md. Signed-off-by: Kir Kolyshkin <[email protected]>
Taken from runc. Signed-off-by: Kir Kolyshkin <[email protected]>
Simpler than that in runc as we don't use vendor here. Signed-off-by: Kir Kolyshkin <[email protected]>
This is so we can mark it as required in PR merge rules. Signed-off-by: Kir Kolyshkin <[email protected]>
Signed-off-by: Kir Kolyshkin <[email protected]>
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.
LGTM
Looks like we need to clean up access on this repo (I shouldn't have a green button!) Pinging maintainers for another review: @AkihiroSuda @cyphar @mrunalp @odinuge @haircommander @dims |
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.
LGTM
@tianon no 🟢 button for me either :) |
This adds some validation for PR as well as running the unit tests.
For a test run, see
Updates #2.