Code review comment for lp:~thekorn/launchpad/make_iperson_ihasbugs

Revision history for this message
Markus Korn (thekorn) wrote :

Thanks for your review Gavin, I tried to address as many points as possible in the recent commits, most importantly I moved the logic to create search parameters into a separate function, added a few more descriptive comments/docstrings, moved the doctest into xx-bug.txt and added unittests for the new function.

review: Needs Resubmitting

« Back to merge proposal