(11:24:01) adeuring: jtv: the changes look good, and thanks a lot for the nice reformatting. just one question: what about a test? (11:24:47) jtv: adeuring: good point — we know that the column is initialized, but it'd be nice to check it for correctness. Let me just see what the existing tests do. -- https://code.launchpad.net/~jtv/launchpad/bug-849683-cloner/+merge/84222 Your team Launchpad code reviewers is requested to review the proposed merge of lp:~jtv/launchpad/bug-849683-cloner into lp:launchpad.
_______________________________________________ Mailing list: https://launchpad.net/~launchpad-reviewers Post to : [email protected] Unsubscribe : https://launchpad.net/~launchpad-reviewers More help : https://help.launchpad.net/ListHelp
