Review: Approve I asked benji about line 35--the technically superfluous urlopen--and he said it was about readability. That's fine by me, given the relative cost of that line in comparison to the test as a whole.
Thanks, Benji! -- https://code.launchpad.net/~benji/launchpad/bug-974617-5/+merge/105893 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
