[Engine-devel] Development process wiki page
Itamar Heim
iheim at redhat.com
Tue Jan 31 09:58:53 UTC 2012
On 01/31/2012 03:03 AM, Eli Mesika wrote:
>
>
> ----- Original Message -----
>> From: "Oved Ourfalli"<ovedo at redhat.com>
>> To: engine-devel at ovirt.org
>> Sent: Monday, January 30, 2012 9:45:47 AM
>> Subject: [Engine-devel] Development process wiki page
>>
>> Hey all,
>>
>> I've wrote a wiki page on the oVirt development process.
>> It contains mostly information on the patch review process, patch
>> submission and some git guidelines.
>>
>> http://www.ovirt.org/wiki/DevProcess
>>
>> I added a link to it from the main wiki page.
>>
>> Your comments are welcome.
>
> my 2 cents : (obvious , but should be in the wiki)
>
> 1) Regarding patch separation, still each commit should not break the build
> 2) When having more than one patch set due to comments and applied fixes add to the commit message V[n] :<changes description> where [n] is the patch set number
I agree we need this, but the problem is we are lacking the [0/nnn]
cover letter email you get for such commentary when using gerrit.
and there is a concept of the commit itself should not include the
history of the patch being reviewed.
so we need to adapt this to gerrit.
maybe a convention that when uploading a new patch set, contributor also
adds a review comment of the changes from previous version (similar to
the cover letter).
I wonder if the git/gerrit utility some adopted from openstack can help
with such a step to allow adding a comment when pushing the patch for
review.
More information about the Engine-devel
mailing list