Code review comment for lp:~vila/bzr/selftest-start-load

Revision history for this message
Vincent Ladeuil (vila) wrote :

>>>>> "martin" == Martin Pool writes:

    martin> Review: Approve

Thanks.

    >> Please, when the diff is so big, put on a more explicit
    >> cover letter saying what you changed...

Talk about the lp bug ruining my joke...

The change had landed on bazaar-vcs.org when I did the merge
request, but apparently it hasn't been mirrored which may partly
explain the problem.

    >>
    >> I guess you mean --load-list?

    martin> OK, I see the real diff is a bit more reasonable.

Yes, the "real" diff was intended to be "obvious" :-/

« Back to merge proposal