Code review comment for lp:~ian-clatworthy/bzr/411413-plugin-disable

Revision history for this message
Ian Clatworthy (ian-clatworthy) wrote :

> Eventually, if a plugin is disabled, it would be nice for this to be reflected
> in 'bzr plugins'.

Agreed. That can easily come later.

> The documentation is unclear about whether you specify eg
> ~/.bazaar/plugins/qbzr or ~/.bazaar/plugins/.

It's the parent. I had hoped the example in the doc made that clear. Matching on the parent is the easiest to code and makes the most sense for plugins that aren't modules. I do wonder whether it's what users would expect though. OTOH, the 90% game here is disabling so maybe it's not worth worrying about too much?

« Back to merge proposal