Review: Approve
This looks great; a couple of comments:

* See above for recommendations against u'' strings and for using print instead 
of returning strings directly in doctests.

* Could you please open a bug (tagged with 'mailman3') on the need to expose an 
API call to look up a member-id given an email address?  I'll make sure to 
expose that in a8.

With the above consideration, I'll approve this change.  Thanks!  And feel free 
to land it.
-- 
https://code.launchpad.net/~flo-fuchs/mailman.client/settings/+merge/56088
Your team Mailman Coders is subscribed to branch lp:mailman.client.
_______________________________________________
Mailman-coders mailing list
[email protected]
http://mail.python.org/mailman/listinfo/mailman-coders

Reply via email to