<div dir="ltr">Hey Darragh, <div><br></div><div>No problem at all. Thanks for the feedback. I'll slowly start removing them from my commits and for future patches.</div><div><br></div><div>Thanks,</div><div>Kien Ha</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Jul 1, 2016 at 10:25 AM, Darragh Bailey <span dir="ltr"><<a href="mailto:daragh.bailey@gmail.com" target="_blank">daragh.bailey@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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"><span class="">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><div><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></span><div>This kind goes outside of what is generally described as <a href="https://wiki.openstack.org/wiki/GitCommitMessages#Information_in_commit_messages" target="_blank">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.<span class="HOEnZb"><font color="#888888"><br><br></font></span></div><span class="HOEnZb"><font color="#888888"><div class="gmail_extra"><br>-- <br><div data-smartmail="gmail_signature">Darragh Bailey<br>"Nothing is foolproof to a sufficiently talented fool"</div>
</div></font></span></div></div>
</blockquote></div><br></div>