launchpad-reviewers team mailing list archive

[Gary Poster <gary.poster@xxxxxxxxxxxxx>]

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.