<html><head><meta http-equiv="Content-Type" content="text/html; charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; line-break: after-white-space;" class=""><div class=""><br class=""></div><div class="">Hi Simon, list et al,</div><div class=""><br class=""></div><div class="">I’ve only contributed a couple of times, but I personally found the checklist invaluable to guide me (and remind me of) what needed to be done in total. In addition, giving folks a checklist that they can actually check off gives us a common set of agreed upon things that’s needed in an MR right in the MR, which is nice to folks.</div><div class=""><br class=""></div><div class="">I wonder if we could reword it to say it’s still a work in progress or words to that effect at the top, and make the system not allow MRs to be built and/or merged unless they edit that text away, as well as have a bot inform them of why this is? :) I like the idea of the system guiding us through the process.</div><div class=""><br class=""></div><div class="">Regards,</div><div class="">Julian</div><div class=""><br class=""></div><div class="">Would it be possible to get our tooling (a bot?) to nudge us if we haven’t changed it?</div><br class=""><div><blockquote type="cite" class=""><div class="">On 24 Feb 2021, at 3:14 am, Simon Peyton Jones via ghc-devs <<a href="mailto:ghc-devs@haskell.org" class="">ghc-devs@haskell.org</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class="WordSection1" style="page: WordSection1; caret-color: rgb(0, 0, 0); font-family: Helvetica; font-size: 12px; font-style: normal; font-variant-caps: normal; font-weight: normal; letter-spacing: normal; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; word-spacing: 0px; -webkit-text-stroke-width: 0px; text-decoration: none;"><p class="MsoNormal" style="margin: 6pt 0cm; font-size: 11pt; font-family: Calibri, sans-serif;">I often see MRs in my inbox that say<o:p class=""></o:p></p><div style="margin: 0cm 0cm 0cm 22.7pt; font-size: 10pt; font-family: "Courier New"; font-weight: bold;" class=""><span lang="EN" class="">Thank you for your contribution to GHC!<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0cm 22.7pt; font-size: 10pt; font-family: "Courier New"; font-weight: bold;" class=""><span lang="EN" class="">Please take a few moments to verify that your commits fulfill the following:<o:p class=""></o:p></span></div><div style="margin: 0cm 0cm 0cm 22.7pt; font-size: 10pt; font-family: "Courier New"; font-weight: bold;" class=""><span lang="EN" class="">[ ] are either individually buildable or squashed<o:p class=""></o:p></span></div><p class="MsoNormal" style="margin: 6pt 0cm; font-size: 11pt; font-family: Calibri, sans-serif;"><o:p class=""> </o:p></p><p class="MsoNormal" style="margin: 6pt 0cm; font-size: 11pt; font-family: Calibri, sans-serif;">This is because the author hasn’t changed the Description of the MR, but rather has left the template text unchanged.<o:p class=""></o:p></p><p class="MsoNormal" style="margin: 6pt 0cm; font-size: 11pt; font-family: Calibri, sans-serif;">As a way to “nudge” authors to give reviewers more information, I suggest replacing the template text with the draft below. Does anyone have any views, for or against?<o:p class=""></o:p></p><p class="MsoNormal" style="margin: 6pt 0cm; font-size: 11pt; font-family: Calibri, sans-serif;">Simon<o:p class=""></o:p></p></div></div></blockquote></div><br class=""></body></html>