[MERGE] Reveal location of .bazaar in `bzr version`

John Arbash Meinel john at arbash-meinel.com
Mon Mar 12 21:20:31 GMT 2007


John Arbash Meinel has voted +1 (conditional).
Status is now: Conditionally approved
Comment:
There is a small PEP8 error in:

self.assertEquals(1,out.count(bzrlib.__version__))

it should be
self.assertEquals(1, out.count(bzrlib.__version__))

(I also prefer assertEqual, but I don't think that is any sort of code 
convention).

To me, monkey patching seems the best way to test the rest. I'll merge 
this with the code cleanup, but it would be nice to get more tests for 
the rest.

For details, see: 
http://bundlebuggy.aaronbentley.com/request/%3C93F40416-66EA-4896-8C2D-8AA5F7F6E050%40bangpath.org%3E



More information about the bazaar mailing list