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

Revision history for this message
Martin Pool (mbp) wrote :

That's a nice feature, thanks for adding it.

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

It would be nice too to have -Ddisable_plugins=qbzr to set it at runtime but that's ok for now.

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

I think setting it to empty to mean disabled is a bit strange but not necessarily wrong.

« Back to merge proposal