[MERGE] Make command discovery entirely callback driven.
Robert Collins
robert.collins at canonical.com
Mon Jun 15 08:07:24 BST 2009
On Mon, 2009-06-15 at 17:03 +1000, Ian Clatworthy wrote:
>
>
> >>>>> +def get_all_cmds():
>
> >>>>> d = _builtin_commands()
> >>>>> if plugins_override:
>
> So I'm trying to say that this variable is still used in the code I
> reviewed, but isn't declared as a parameter any more. Maybe you've
> changed the code since ...
Oh! I get it now. Changed my mind in some code changes; I'll put the
parameter back as the easiest way to fix.
> > remaining issues:
> > - do we need to warn people off from using the hooks
>
> We don't need to warn them off - just let them know that they aren't
> *required* to use them to register a command in a plugin.
Ok. I'm not sure how best to get the message across though:- do you
think just mentioning it in the docs for (say) get_command is enough?
The infrastructure for hook docs doesn't have a slot for 'conceptual
docs' at the moment.
Rob
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: This is a digitally signed message part
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20090615/19632bbd/attachment.pgp
More information about the bazaar
mailing list