[PATCH] Re: FEATURE REQ: Add option --dry-run to 'add'

Martin Pool martinpool at gmail.com
Mon Sep 19 08:28:41 BST 2005


On 17/09/05, Michael Ellerman <michael at ellerman.id.au> wrote:
> Hi guys,
> 
> Here's a patch to do --dry-run for add. It's a bit busy, as I've changed the
> API for smart_add() and hence all it's callers have to be updated, but I
> think this is the cleanest way to implement this. Otherwise we end up with
> smart_add() taking 15 parameters.
> 
> It includes tests for the action callbacks, as well as a blackbox test of "bzr
> add --dry-run .

That looks reasonable. I like the refactoring into an action callback.
 I'm going to do an 0.0.8 release with what's already been pulled in
and then add this.

-- 
Martin




More information about the bazaar mailing list