<div dir="ltr">Mea Culpa. Mea maxima culpa.<div><br></div><div>Thanks for the link, Colin. There are a couple of related issues that I had noticed along the way:</div><div><br></div><div>1) Convention (and possibly some koha programming standard) says that the bug number be included in the summary line of the commit message. Somewhere along the line, I assumed that this was an automated thing, so I left them off to avoid duplication :-/ (I'm in the process of fixing those). If we did want to automate this, where would we put it in the process?</div><div><br></div><div>2) Knowing which branches a patch has been pushed to is a pain-point for those of us supporting Koha [I know that this is a bit off topic, because this is *not* information that would be stored in the commit message, but bear with me...]. Typically, QA will announce that a patch has been pushed to master, and then the release maintainers for the supported releases will apply the patch to the point releases of the supported Koha version, if applicable. This can cause issues for support because we don't necessarily know what version master was at when the bug was pushed. You can get some sense of this from the date the bug was pushed, and look up release dates, but that's not a straightforward process by any means. I'm not a QA guy, but I assume that we have some release management tool... could we automate the process so that it posted the Koha version number to bugzilla when a bug is pushed to master?</div><div><br></div><div>Thanks,</div><div><br></div><div>--Barton</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 23, 2015 at 5:47 AM, Jonathan Druart <span dir="ltr"><<a href="mailto:jonathan.druart@bugs.koha-community.org" target="_blank">jonathan.druart@bugs.koha-community.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">I plead guilty :)<br>
I already try to follow these instructions but I tend to forget!<br>
<br>
Do you have some recent examples?<br>
<br>
Cheers,<br>
Jonathan<br>
<div class="HOEnZb"><div class="h5"><br>
2015-09-23 9:29 GMT+01:00 Colin Campbell <<a href="mailto:colin.campbell@ptfs-europe.com">colin.campbell@ptfs-europe.com</a>>:<br>
> Hi,<br>
> can I humbly suggest that we should take a couple of secs to make the<br>
> commit messages for submitted patches a bit more useful. When looking at<br>
> the history of the code its useful if the summary gives a brief<br>
> indication of what has been changed or added by the commit. Good<br>
> summaries can save a lot of frustrated time wasting when trying to track<br>
> down where a change has been introduced.<br>
> Although we flag up the bugzilla bug in the summary, just echoing the<br>
> bug summary is nearly always counter-productive. The commit should<br>
> indicate the change introduced not the bug that prompted it. Once the<br>
> patch is applied the original bug is "background reading" not the main<br>
> event. It is also misleading if the commit summary says "X is broken"<br>
> then when that is extracted into the release notes or the gitlog it<br>
> looks like the commit introduces the breakage of x not vice-versa. It is<br>
> surely better to have the commit say something along the lines of<br>
> "Fix broken behaviour in X", which makes it sound a bit more like the<br>
> desirable feature it no doubt is.<br>
><br>
> A long standing succinct post on commit messages is<br>
><br>
> <a href="http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html" rel="noreferrer" target="_blank">http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html</a><br>
><br>
> Colin<br>
> (Author of some of the world's worst commit messages)<br>
><br>
> --<br>
> Colin Campbell<br>
> Chief Software Engineer,<br>
> PTFS Europe Limited<br>
> Content Management and Library Solutions<br>
> <a href="tel:%2B44%20%280%29%20800%20756%206803" value="+448007566803">+44 (0) 800 756 6803</a> (phone)<br>
> <a href="tel:%2B44%20%280%29%207759%20633626" value="+447759633626">+44 (0) 7759 633626</a> (mobile)<br>
> <a href="mailto:colin.campbell@ptfs-europe.com">colin.campbell@ptfs-europe.com</a><br>
> skype: colin_campbell2<br>
><br>
> <a href="http://www.ptfs-europe.com" rel="noreferrer" target="_blank">http://www.ptfs-europe.com</a><br>
> _______________________________________________<br>
> Koha-devel mailing list<br>
> <a href="mailto:Koha-devel@lists.koha-community.org">Koha-devel@lists.koha-community.org</a><br>
> <a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
> website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
> git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
> bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a><br>
_______________________________________________<br>
Koha-devel mailing list<br>
<a href="mailto:Koha-devel@lists.koha-community.org">Koha-devel@lists.koha-community.org</a><br>
<a href="http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel" rel="noreferrer" target="_blank">http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-devel</a><br>
website : <a href="http://www.koha-community.org/" rel="noreferrer" target="_blank">http://www.koha-community.org/</a><br>
git : <a href="http://git.koha-community.org/" rel="noreferrer" target="_blank">http://git.koha-community.org/</a><br>
bugs : <a href="http://bugs.koha-community.org/" rel="noreferrer" target="_blank">http://bugs.koha-community.org/</a><br>
</div></div></blockquote></div><br></div>