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

Michael Ellerman michael at ellerman.id.au
Thu Feb 16 01:31:50 GMT 2006


OK, all fixed and uploaded. Pull at your convenience.

cheers

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:
> >> 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" ??
> >
>
> Yes.
>
> >> 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
> >
>
> John
> =:->
>
>
>
>




More information about the bazaar mailing list