<div dir="ltr"><div class="gmail_extra">Hey Darragh and Kien,</div><div class="gmail_extra"><br></div><div class="gmail_extra">What I proposed is to create a etherpad/spec/blueprint to cover these changes of using the convert to xml function in JJB, and include it in the commit message as reference to keep tracking the process, so if someone else would like to contribute, it will be easy to find it.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Yes, include the document link into the commit message is not a good commit message, I agree to keep the commits describing the functional reason.</div><div class="gmail_extra"><br></div><div class="gmail_extra">Thanks,</div><div class="gmail_extra">Vincent</div><div class="gmail_extra"><br><div class="gmail_quote">2016-07-01 22:25 GMT+08:00 Darragh Bailey <span dir="ltr"><<a href="mailto:daragh.bailey@gmail.com" target="_blank">daragh.bailey@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);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>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-width:1px;border-left-style:solid;border-left-color: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-width:1px;border-left-style:solid;border-left-color: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><font color="#888888"><br><br></font></span></div><span><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></div>