Bazaar

Code review comment for lp:~mathepic/bzr/remove-tree-multi

  1. remove-tree-multi
  2. Merge into bzr.dev
Revision history for this message
Andrew Bennetts (spiv) wrote :

This patch broke the existing blackbox tests for test_remove_tree. It turns out that you accidentally moved some code inside the "if not force:" block. I've fixed that and added a test. My branch is at lp:~spiv/bzr/remove-tree-multi-253137

All that's left is for you to sign the contributor agreement: <http://www.canonical.com/contributors>

review: Needs Fixing

« Back to merge proposal