<div dir="ltr"><div class="gmail_default" style="font-size:small">Hi all,<br clear="all"></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">I find it's hard sometimes to handle situations in code-review, something likes solving conflicts while not upsetting developers, or suggesting a change to a patchset while still encouraging the committer, etc. I know there are already documents that guide us on how to do a code-review [2] and even projects develope their own procedures but I find they're more about technical issues rather than human communication. Currently reading Google's code-review practices [1] give me some inspiration to develop more human-centric code-review guidelines for OpenStack projects. IMO, it could be a great way to help project teams develop stronger relationship as well as encouraging newcomers. When the document is finalized, I then encourage PTLs to refer to that document in the project's docs.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Let me know what you think and I will put a patchset after one or two weeks.</div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">[1] <a href="https://google.github.io/eng-practices/review/">https://google.github.io/eng-practices/review/</a><br></div><div class="gmail_default" style="font-size:small">[2] <a href="https://docs.openstack.org/project-team-guide/review-the-openstack-way.html">https://docs.openstack.org/project-team-guide/review-the-openstack-way.html</a><br></div><div class="gmail_default" style="font-size:small">[3] <a href="https://docs.openstack.org/doc-contrib-guide/docs-review.html">https://docs.openstack.org/doc-contrib-guide/docs-review.html</a><br></div><div class="gmail_default" style="font-size:small">[4] <a href="https://docs.openstack.org/nova/rocky/contributor/code-review.html">https://docs.openstack.org/nova/rocky/contributor/code-review.html</a><br></div><div class="gmail_default" style="font-size:small">[5] <a href="https://docs.openstack.org/neutron/pike/contributor/policies/code-reviews.html">https://docs.openstack.org/neutron/pike/contributor/policies/code-reviews.html</a><br></div><div><br></div><div><br></div><div><div class="gmail_default" style="font-size:small">Bests,</div><br></div>-- <br><div dir="ltr" class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div><b style="font-size:small;color:rgb(51,51,51)">Trinh Nguyen</b><br></div><div><u style="font-size:12.8px;color:rgb(0,0,0)"><a href="https://www.edlab.xyz" target="_blank">www.edlab.xyz</a></u><br></div><div><br></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>