[MERGE] Hook documentation

Andrew Bennetts andrew.bennetts at canonical.com
Mon Oct 20 12:45:51 BST 2008


Michael Ernst wrote:
[...]
> Andrew-
> 
> Thanks for your comments and clarifications on my patch; I appreciate them.
> 
> I have incorporated them all in the attached merge directive, except for
> one:
> 
> > > The first parameter ``'post_push'``
> > 
> > I'm not sure that the markup of $,1r|``'post_push'``$,1r} makes sense $,1rt why include
> > single quotes here, inside the monospaced text?
> 
> This formatting is retained from before my patch.  I presume it's because
> that's the actual parameter is a single-quoted 9-character string, and the
> text is referring to the actual argument.

Ah, right.  I'm not sure why I thought that was odd, it clearly makes
sense now :)

bb:approve

-Andrew.




More information about the bazaar mailing list