<div dir="ltr"><div class="gmail_default" style="font-family:verdana,sans-serif;font-size:small">All sounds very sensible to me.</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Oct 8, 2019 at 2:17 PM Ben Gamari <<a href="mailto:ben@well-typed.com">ben@well-typed.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">tl;dr. I would like feedback on a few proposed changes [1] to our merge<br>
       request workflow.<br>
<br>
<br>
Hello everyone,<br>
<br>
Over the past six months I have been monitoring the operation of our<br>
merge request workflow, which arose rather organically in the wake of<br>
the initial move to GitLab. While it works reasonably well, there is<br>
clearly room for improvement:<br>
<br>
  * we have no formal way to track the status of in-flight merge<br>
    requests (e.g. for authors to mark an MR as ready for review or<br>
    reviewers to mark work as ready for merge)<br>
<br>
  * merge requests still at times languish without review<br>
<br>
  * the backport protocol is somewhat error prone and requires a great<br>
    deal of attention to ensure that patches don't slip through the<br>
    cracks<br>
<br>
  * there is no technical mechanism to prevent that under-reviewed<br>
    patches from being merged (either intentionally or otherwise) to<br>
    `master`<br>
<br>
To address this I propose [1] a few changes to our workflow:<br>
<br>
  1. Define explicit phases of the merge request lifecycle,<br>
     systematically identified with labels. This will help to make it<br>
     clear who is responsible for a merge request at every stage of its<br>
     lifecycle.<br>
<br>
  2. Make it clear that it is the contributor's responsibility to<br>
     identify reviewers for their merge requests.<br>
<br>
  3. Institute a final pre-merge sanity check to ensure that<br>
     patches are adequately reviewed, documented, tested, and have had<br>
     their ticket and MR metadata updated.<br>
<br>
Note that this is merely a proposal; I am actively seeking input from<br>
the developer community. Do let me know what you think.<br>
<br>
Cheers,<br>
<br>
- Ben<br>
<br>
<br>
[1] <a href="https://gitlab.haskell.org/ghc/ghc/wikis/proposals/merge-request-workflow" rel="noreferrer" target="_blank">https://gitlab.haskell.org/ghc/ghc/wikis/proposals/merge-request-workflow</a><br>
_______________________________________________<br>
ghc-devs mailing list<br>
<a href="mailto:ghc-devs@haskell.org" target="_blank">ghc-devs@haskell.org</a><br>
<a href="http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs" rel="noreferrer" target="_blank">http://mail.haskell.org/cgi-bin/mailman/listinfo/ghc-devs</a><br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="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 dir="ltr"><div style="letter-spacing:normal;padding:10px 0px 10px 10px;width:300px;max-width:300px;color:rgb(26,40,77);font-size:12px"><div style="color:rgb(33,33,33);font-size:13px"><div><div><div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr"><div style="padding:10px 0px 10px 10px;width:300px;max-width:300px;color:rgb(26,40,77);font-size:12px"><div style="color:rgb(34,34,34);font-size:small"><b><font color="#000000">Shayne Fletcher</font></b><br></div><div style="font-size:small;color:rgb(80,0,80)"><font size="1"><font color="#444444">Language Engineer </font><font color="#0b5394"><b>/</b></font><font color="#444444"> +1 917 699 7663</font><br></font></div><font size="1" style="color:rgb(34,34,34)"><a href="https://digitalasset.com/" style="color:rgb(17,85,204)" target="_blank"><b><font color="#3d85c6">Digital Asset</font></b></a><font style="color:rgb(26,40,77)">, creators of </font><b><a href="https://daml.com/" style="color:rgb(17,85,204)" target="_blank"><font color="#3d85c6">DAML</font></a></b></font><br></div></div></div></div></div></div></div></div><div style="font-family:sans-serif;color:rgb(33,33,33);font-size:13px"></div></div></div></div></div></div></div></div></div></div></div></div></div></div></div>

<br>
<span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:12.8000001907349px;background-color:rgb(255,255,255)">This message, and any attachments, is for the intended recipient(s) only, may contain information that is privileged, confidential and/or proprietary and subject to important terms and conditions available at </span><a href="http://www.digitalasset.com/emaildisclaimer.html" style="color:rgb(17,85,204);font-family:arial,sans-serif;font-size:12.8px;background-color:rgb(255,255,255)" target="_blank">http://www.digitalasset.<wbr>com/emaildisclaimer.html</a><span style="color:rgb(34,34,34);font-family:arial,sans-serif;font-size:12.8000001907349px;background-color:rgb(255,255,255)">. If you are not the intended recipient, please delete this message.</span>