-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
Update contribution guidelines #4619
Update contribution guidelines #4619
Conversation
ea07dd3
to
9de3bc4
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #4619 +/- ##
=======================================
Coverage 82.08% 82.08%
=======================================
Files 255 255
Lines 31256 31256
=======================================
Hits 25658 25658
Misses 5598 5598
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
Thanks for the changes @roypat. Overall LGTM but i have a small suggestion which i passed on in the review.
9de3bc4
to
abc3bde
Compare
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, thanks for the changes.
PRs can only get merged if it has at least 2 approvals from maintainers, not 1 approval from a maintainer and 1 approval from someone else (which doesn't make sense). Additionally, all review comments need to be addressed. Signed-off-by: Patrick Roy <roypat@amazon.co.uk>
Sudan has left the team. Signed-off-by: Patrick Roy <roypat@amazon.co.uk>
Clarifies that we want to avoid unwraping in favor of propagating errors. Closes firecracker-microvm#808 Signed-off-by: Patrick Roy <roypat@amazon.co.uk>
abc3bde
to
3eddd50
Compare
Changes
.unwrap()
Reason
#808
License Acceptance
By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license. For more information on following Developer
Certificate of Origin and signing off your commits, please check
CONTRIBUTING.md
.PR Checklist
PR.
CHANGELOG.md
.TODO
s link to an issue.contribution quality standards.
rust-vmm
.