Can we set a location to the Authoritative HACKING.rst?<br><br>There are fundamental and conflicting differences between the HACKING.rst in some of the projects.<br><br>-Matt<br><br><div class="gmail_quote">On Thu, Jun 28, 2012 at 3:15 AM, Daniel P. Berrange <span dir="ltr"><<a href="mailto:berrange@redhat.com" target="_blank">berrange@redhat.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Thu, Jun 28, 2012 at 12:01:10PM +0200, Thierry Carrez wrote:<br>
> Daniel P. Berrange wrote:<br>
> > [...]<br>
> > In other words, when reviewing a change in Gerrit, do not simply look at<br>
> > the correctness of the code. Review the commit message itself and request<br>
> > improvements to its content. Look out for commits which are mixing multiple<br>
> > logical changes and require the submitter to split them into separate commits.<br>
> > Ensure whitespace changes are not mixed in with functional changes. Ensure<br>
> > no-op code refactoring is done separately from functional changes. And so<br>
> > on.<br>
> > [...]<br>
><br>
> Nice work, and agreed on all points ! I particularly hate the<br>
> single-line "Fixes bug 1234566"-type commit messages.<br>
><br>
> Is there a way a concise version of this advice could find its way into<br>
> HACKING.rst ? And/Or into <a href="http://wiki.openstack.org/ReviewChecklist" target="_blank">http://wiki.openstack.org/ReviewChecklist</a> ?<br>
<br>
</div>Sure, MarkMc suggested to me that I put this doc up on the wiki somewhere.<br>
I'll do that and then submit a concise version for HACKING.rst and<br>
the ReviewChecklist page, with a cross-reference to the full thing.<br>
<br>
Regards,<br>
<div class="im HOEnZb">Daniel<br>
--<br>
|: <a href="http://berrange.com" target="_blank">http://berrange.com</a>      -o-    <a href="http://www.flickr.com/photos/dberrange/" target="_blank">http://www.flickr.com/photos/dberrange/</a> :|<br>
|: <a href="http://libvirt.org" target="_blank">http://libvirt.org</a>              -o-             <a href="http://virt-manager.org" target="_blank">http://virt-manager.org</a> :|<br>
|: <a href="http://autobuild.org" target="_blank">http://autobuild.org</a>       -o-         <a href="http://search.cpan.org/%7Edanberr/" target="_blank">http://search.cpan.org/~danberr/</a> :|<br>
|: <a href="http://entangle-photo.org" target="_blank">http://entangle-photo.org</a>       -o-       <a href="http://live.gnome.org/gtk-vnc" target="_blank">http://live.gnome.org/gtk-vnc</a> :|<br>
<br>
</div><div class="HOEnZb"><div class="h5">_______________________________________________<br>
Mailing list: <a href="https://launchpad.net/%7Eopenstack" target="_blank">https://launchpad.net/~openstack</a><br>
Post to     : <a href="mailto:openstack@lists.launchpad.net">openstack@lists.launchpad.net</a><br>
Unsubscribe : <a href="https://launchpad.net/%7Eopenstack" target="_blank">https://launchpad.net/~openstack</a><br>
More help   : <a href="https://help.launchpad.net/ListHelp" target="_blank">https://help.launchpad.net/ListHelp</a><br>
</div></div></blockquote></div><br>