Code review and commits to the official repository

[William Hubbs]

On Wed, Aug 11, 2010 at 03:51:38PM +0200, Tomas Cerha wrote:
> Dne 10.8.2010 17:39, Chris Brannon napsal(a):
> > This sounds good.
> > How about signed-off-by lines?  Should the reviewer add one before
> > committing?  I like them, myself, because it is obvious who reviewed
> > the patch.
> 
> Good point.  I haven't used that before, but it looks good to me.  Maybe
> the "Reviewed-by" tag would be more appropriate for us, according to:
> 
> http://gerrit.googlecode.com/svn/documentation/2.0/user-signedoffby.html

I'm not sure how you do a Reviewed-by tag in git, but Signed-off-by is
as easy as adding a -s option to "git commit" or "git am".

William

-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: 
<http://lists.freebsoft.org/pipermail/speechd/attachments/20100811/06bab01a/attachment.pgp>