Launchpad itself

Code review comment for lp:~adeuring/launchpad/bug-834303

  1. bug-834303
  2. Merge into devel
Revision history for this message
Richard Harding (rharding) wrote :

Looks good, my only comment is that I don't think you need the set() in line 30 since I don't think the subscriber ids should have dupes in there.

review: Approve (code*)

« Back to merge proposal