<div dir="ltr"><div>Hi Kien,<br><br><br></div>I missed this email, but spotted this appearing in recent commits.<br><br><div><div class="gmail_extra"><br><div class="gmail_quote">On 13 June 2016 at 03:28, Kien Ha <span dir="ltr"><<a href="mailto:kienha9922@gmail.com" target="_blank">kienha9922@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"></div><div class=""><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 11, 2016 at 11:12 PM, Dong Ma <span dir="ltr"><<a href="mailto:winterma.dong@gmail.com" target="_blank">winterma.dong@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr">Hello Kien Ha,<div><br></div><div>Thanks for the contribution to the Jenkins job builder projects, have one comment here, how about add your proposal document link or create a new etherpad into the commit message of each patches as reference to keep tracking the process.</div><div><br></div><div>Vincent</div></div></blockquote></div></div></div></div></blockquote><div><br></div><div>This kind goes outside of what is generally described as <a href="https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages">https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages</a><br></div><br></div><br></div><div class="gmail_extra">The update to the mailing list is more than enough, let's keep the commits describing the functional reason for the change, or if there is a separate proposal spec/blueprint that covers changing to use the convert to xml function in JJB, changes relevant can include a reference, otherwise lets keep the commit messages focused on the change itself.<br><br></div><div class="gmail_extra"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature">Darragh Bailey<br>"Nothing is foolproof to a sufficiently talented fool"</div>
</div></div></div>