Message178948
| Author |
zach.ware |
| Recipients |
brett.cannon, ezio.melotti, pjenvey, zach.ware |
| Date |
2013年01月03日.15:19:33 |
| SpamBayes Score |
-1.0 |
| Marked as misclassified |
Yes |
| Message-id |
<1357226373.83.0.775140068045.issue16835@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
> Thanks for the review, new patch attached.
You're quite welcome. Is there anything I've missed in the process of reviewing itself? This is the first time I've reviewed a patch here...
I did miss another nit in the prose, though; "the tests methods" in the first line isn't quite right, but I can't decide if it should be "the test's methods" (singular possessive), "the tests' methods" (plural possessive), or "test methods" (non-specific, non-possessive). Any of the three that better gets your point across makes me happy :)
>> This should either have another newline inbetween or be reflowed.
>> Either one paragraph or two makes sense to me, but I can't tell which
>> way you actually meant
>
> I meant one and half :)
> I guess in HTML that would have been a <br> inside the <p>, but I don't
> think it really matters if it's rendered as a single paragraph.
I see. As is works for me if it works for you, though I might lean towards rounding it up to 2 paragraphs :) |
|
History
|
|---|
| Date |
User |
Action |
Args |
| 2013年01月03日 15:19:33 | zach.ware | set | recipients:
+ zach.ware, brett.cannon, pjenvey, ezio.melotti |
| 2013年01月03日 15:19:33 | zach.ware | set | messageid: <1357226373.83.0.775140068045.issue16835@psf.upfronthosting.co.za> |
| 2013年01月03日 15:19:33 | zach.ware | link | issue16835 messages |
| 2013年01月03日 15:19:33 | zach.ware | create |
|