<div dir="ltr"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Mar 19, 2021 at 2:17 AM Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Signed-off-by: Petr Vorel <<a href="mailto:pvorel@suse.cz" target="_blank">pvorel@suse.cz</a>><br>
---<br>
 doc/maintainer-patch-review-checklist.txt | 53 +++++++++++++++++++++++<br>
 1 file changed, 53 insertions(+)<br>
 create mode 100644 doc/maintainer-patch-review-checklist.txt<br>
<br>
diff --git a/doc/maintainer-patch-review-checklist.txt b/doc/maintainer-patch-review-checklist.txt<br>
new file mode 100644<br>
index 000000000..2587287aa<br>
--- /dev/null<br>
+++ b/doc/maintainer-patch-review-checklist.txt<br>
@@ -0,0 +1,53 @@<br>
+# Maintainer Patch Review Checklist<br>
+<br>
+Patchset should be tested locally and ideally also in maintainer's fork in<br>
+<a href="https://travis-ci.org/%5BTravis" rel="noreferrer" target="_blank">https://travis-ci.org/[Travis</a> CI].<br>
+<br>
+NOTE: Travis does only build testing, passing the CI means only that the<br>
+      test compiles fine on variety of different distributions and<br>
+      releases.<br>
+<br>
+The test should be executed at least once locally and should PASS as well.<br>
+<br>
+Commit messages should have<br>
+<br>
+* Author's `Signed-off-by` tag<br>
+* Committer's `Reviewed-by` or `Signed-off-by` tag<br></blockquote><div><br></div><div><div class="gmail_default" style="font-size:small">Or adding `Acked-by:` also acceptable?</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">I'm not sure what's the difference between Acked-by and Reviewed-by,</div><div class="gmail_default" style="font-size:small">but personally feel the Acked-by has more confidence in the reviewing.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">In my mind,</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">`Acked-by` means:</div><div class="gmail_default" style="font-size:small">code review OK, compiled OK, executed OK, has confidence it OK for all arches</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">`Reviewed-by` means:</div><div class="gmail_default" style="font-size:small">code review OK, compiled and executed OK just locally or optional, still<span style="color:rgb(51,51,51);font-size:14px"> room for improvement/discussion</span></div></div></div><div><br></div>-- <br><div dir="ltr"><div dir="ltr"><div>Regards,<br></div><div>Li Wang<br></div></div></div></div>