Review: Approve This looks good to me. As a minor point, you can probably drop several places where you're asserting the length of an array, as you then assert the contents equal a particular list. The length assertion is implicit in comparison of contents. -- https://code.launchpad.net/~sinzui/launchpad/empty-attchments-message/+merge/127869 Your team Launchpad code reviewers is subscribed to branch 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
