<div dir="ltr"><div>Hi Jani,</div><div><br></div><div>Apologies on this - I understand the review part as well, you're right.</div><div><br></div><div>Just to give you context: This patch was a quick response to Daniel's ask to update the documentation over IRC. I didn't know the process for this, and assuming gitlab, created a PR. By the time I could offer Daniel to send a patch, he had merged the PR.</div><div><br></div><div>Irrespective, I will take care going forward. Apologies again.</div><div><br></div><div>Best,</div><div>Sumit.<br></div><div><br></div><div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jan 13, 2021 at 7:01 PM Jani Nikula <<a href="mailto:jani.nikula@intel.com">jani.nikula@intel.com</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"><br>
Hi Sumit, I just noticed commit 3ed2489ca753 ("Update repositories.rst<br>
for current drm-misc maintainers") in the maintainer-tools repository.<br>
<br>
The change appears to have been pushed without first sending the patch<br>
to the dim-tools mailing list, and apparently without review.<br>
<br>
Regardless of how trivial or minor the change appears to be, review is<br>
mandatory. For that to happen, posting to the mailing list is mandatory.<br>
<br>
Please see CONTRIBUTING.rst in the repository.<br>
<br>
---<br>
<br>
As to the change, the review, given the chance, would have been that the<br>
change is not enough. We still seem to have Sean's email in the dim<br>
script.<br>
<br>
<br>
BR,<br>
Jani.<br>
<br>
<br>
-- <br>
Jani Nikula, Intel Open Source Graphics Center<br>
</blockquote></div>