Code review comment for lp:~bac/launchpad/bug-154587

Revision history for this message
Michael Nelson (michael.nelson) wrote :

Thanks for all the tidying up too Brad.

This looks great, and I've approved it, but out of interest, was wondering if there was a reason why you added to the doctest rather than putting this in lp.registry.browser.tests.test_person_view?

Thanks!

review: Approve (code)

« Back to merge proposal