Launchpad itself

Code review comment for lp:~sinzui/launchpad/person-merge-job-4

  1. person-merge-job-4
  2. Merge into devel
Revision history for this message
Benji York (benji) wrote :

This branch looks good.

One thing you might want to look at, I /think/ there is a hole in the
testing around the "is_merge_pending" property. I don't see that there
are any tests that the property is ever true.

review: Approve (code)

« Back to merge proposal