[openstack-dev] Code review study
dolph.mathews at gmail.com
Thu Aug 15 14:46:07 UTC 2013
On Thu, Aug 15, 2013 at 7:57 AM, Christopher Yeoh <cbkyeoh at gmail.com> wrote:
> On Thu, Aug 15, 2013 at 9:54 PM, Daniel P. Berrange <berrange at redhat.com>wrote:Commit message quality has improved somewhat since I first wrote &
> that page, but there's definitely still scope to improve things further.
>> it really needs is for more reviewers to push back against badly written
>> commit messages, to nudge authors into the habit of being more verbose in
>> their commits.
> Agreed. There is often "what" and sometimes "why", but not very often
> "how" in commit messages.
Beyond the one line summary (which *should* describe "what" changed),
describing "what" changed in the commit message is entirely redundant with
the commit itself.
> OpenStack-dev mailing list
> OpenStack-dev at lists.openstack.org
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the OpenStack-dev