[MERGE] Bug fix for bzr log -r ..x

Michael Ellerman michael at ellerman.id.au
Thu Feb 16 00:08:13 GMT 2006


On 2/16/06, John A Meinel <john at arbash-meinel.com> wrote:
> Michael Ellerman wrote:
> > On 2/16/06, John A Meinel <john at arbash-meinel.com> wrote:
> >>> Pullable from http://michael.ellerman.id.au/files/bzr/mpe
> Well, here is a review of the changes.
>
> in test_help.py you have the name of the class as TestLog, and you need
> an extra newline before the class definition.

Doh, copy and paste strikes again. Why do I need an extra newline
before the class def? Is that PEP8, "must have two blank lines between
import and class def" ??

> I didn't even realize 'bzr ?' would work. Do we want this alias?

Yeah me neither :) - I think it's kinda nice, and pretty harmless.

cheers




More information about the bazaar mailing list