Hi Aaron,
The code looks good. I assume there are no pagetests for this? (I'm sure ec2 will catch them if there are.)
I am surprised we do not already have a basic test helper like the one you coded. Is there a duplicate helper, or just a lot of code duplication?
Looks good, r=mars
Maris
« Back to merge proposal
Hi Aaron,
The code looks good. I assume there are no pagetests for this? (I'm sure ec2 will catch them if there are.)
I am surprised we do not already have a basic test helper like the one you coded. Is there a duplicate helper, or just a lot of code duplication?
Looks good, r=mars
Maris