<p dir="ltr">I agree Duncan.</p>
<p dir="ltr">I think the commit message is one of the most important parts of a commit.   If the message is not useful, the code shouldn't go in.</p>
<p dir="ltr">Jay Bryant</p>
<div class="gmail_quote">On Jun 22, 2014 1:51 PM, "Duncan Thomas" <<a href="mailto:duncan.thomas@gmail.com">duncan.thomas@gmail.com</a>> wrote:<br type="attribution"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
On 22 June 2014 14:41, Amrith Kumar <<a href="mailto:amrith@tesora.com">amrith@tesora.com</a>> wrote:<br>
> In addition to making changes to the hacking rules, why don't we mandate also<br>
> that perceived problems in the commit message shall not be an acceptable<br>
> reason to -1 a change.<br>
<br>
-1.<br>
<br>
There are some /really/ bad commit messages out there, and some of us<br>
try to use the commit messages to usefully sort through the changes<br>
(i.e. I often -1 in cinder a change only affects one driver and that<br>
isn't clear from the summary).<br>
<br>
If the perceived problem is grammatical, I'm a bit more on board with<br>
it not a reason to rev a patch, but core reviewers can +2/A over the<br>
top of a -1 anyway...<br>
<br>
> Would this improve the situation?<br>
<br>
Writing better commit messages in the first place would improve the situation?<br>
<br>
_______________________________________________<br>
OpenStack-dev mailing list<br>
<a href="mailto:OpenStack-dev@lists.openstack.org">OpenStack-dev@lists.openstack.org</a><br>
<a href="http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev" target="_blank">http://lists.openstack.org/cgi-bin/mailman/listinfo/openstack-dev</a><br>
</blockquote></div>