Code review comment for lp:~msapiro/mailman/bug_949924

Revision history for this message
Mark Sapiro (msapiro) wrote :

I added a unit test. It probably needs some cleanup as I still don't really understand the unittest framework.

I renamed the approve.rst doctest to approved.rst for consistency.

I added a bug fix note to NEWS.

« Back to merge proposal