merge/review process

Martin Pool mbp at sourcefrog.net
Fri Dec 9 01:56:30 GMT 2005


On  9 Dec 2005, Robert Collins <robertc at robertcollins.net> wrote:
> On Thu, 2005-12-08 at 07:39 -0600, John Arbash Meinel wrote:
> > Martin Pool wrote:
> > > I'd like to suggest a system for reviewing code for integration.
> > > 
> > > If you'd like to propose a change, please post a patch, bzr changeset,
> > > or link to a branch.  Please put either '[patch]' or '[merge]' in the
> > > subject so we can pick them out, and include some text explaining the
> > > change.  Remember to put an update to the NEWS file in your diff, if it makes
> > > any changes visible to users or plugin developers.
> 
> Lets reserve [merge] to mean 'please merge this.' And [patch] to mean
> 'there is code in this email' -> not 'please merge this code'. 

OK, sounds good.  So anything that wants to be reviewed for merge should
say [patch] - and preferably contain a diff even if you want me/us to
merge from a branch.

-- 
Martin
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051209/0e489375/attachment.pgp 


More information about the bazaar mailing list