<div dir="ltr"><div class="gmail_quote"><div>There are still people who don't look at the merge requests in gitlab yet, like me. :) I've noticed there are fewer emails... I'll switch after you guys figure out whether MRs are better.<br></div><div><br></div><div>Marek<br></div><div dir="ltr"><br></div><div dir="ltr">On Mon, Jan 14, 2019 at 7:36 AM Daniel Stone <<a href="mailto:daniel@fooishbar.org" target="_blank">daniel@fooishbar.org</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">Hi,<br>
<br>
On Fri, 11 Jan 2019 at 17:05, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>> wrote:<br>
>  5. There's no way with gitlab for Reviewed-by tags to get automatically applied as part of the merging process.  This makes merging a bit more manual than it needs to be but is really no worse than it was before.<br>
<br>
I'm still on the side of not seeing the value in them. Most of the<br>
time when I go to pursue someone who reviewed a commit, I'll go to see<br>
what came up in review anyway. Maybe someone had the same comment<br>
which was found to be not applicable or otherwise explained away.<br>
Reviewed-by and Acked-by are also pretty lossy anyway, and freeform<br>
text descriptors in a comment can much better capture the intent (e.g.<br>
'I'm strongly OK with the driver changes and weakly OK with the core<br>
changes as it's not really my area of expertise').<br>
<br>
In other projects, we looked for ways to apply the tags and ended up<br>
concluding that they didn't bring enough value to make it worthwhile.<br>
I don't know if that holds for Mesa, but it would be better to start<br>
with an actual problem statement - what value does R-b bring and how?<br>
- then look at ways to solve that problem, rather than just very<br>
directly finding a way to insert that literal text string into every<br>
commit message.<br>
<br>
FWIW, if you go to<br>
<a href="https://gitlab.freedesktop.org/mesa/mesa/commit/SHA1" rel="noreferrer" target="_blank">https://gitlab.freedesktop.org/mesa/mesa/commit/SHA1</a> then you get a<br>
hyperlink from the web UI which points you to the MR. The API to do<br>
this is pretty straightforward and amenable to piping through jq:<br>
<a href="https://docs.gitlab.com/ce/api/commits.html#list-merge-requests-associated-with-a-commit" rel="noreferrer" target="_blank">https://docs.gitlab.com/ce/api/commits.html#list-merge-requests-associated-with-a-commit</a><br>
<br>
Cheers,<br>
Daniel<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org" target="_blank">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</blockquote></div></div>