Code review comment for lp:~lifeless/launchpad/databasefixture

Revision history for this message
Robert Collins (lifeless) wrote :

On Wed, Oct 20, 2010 at 4:48 AM, Jonathan Lange <email address hidden> wrote:
> It's hard to see if anything much changed in the LibrarianLayer, since diff completely removes and then adds it.

Yeah, it would be nice to show that better. Nothing did.

> The rest of the changes look sane to me, but I haven't gone through them with a tooth comb looking for correctness issues.

Thats fine, thanks.

-Rob

« Back to merge proposal