Message123866
Patch looks good. Can you add tests for the new functionality? (This is listed in the link I gave you :)
Note: this code
if 'aliases' in kwargs:
aliases = kwargs.pop('aliases')
else:
aliases = ()
can be shortened to
aliases = kwargs.pop('aliases', ()) |
|
Date |
User |
Action |
Args |
2010-12-13 09:18:06 | eric.araujo | set | recipients:
+ eric.araujo, bethard, asampson |
2010-12-13 09:18:06 | eric.araujo | set | messageid: <1292231886.43.0.343556393224.issue9234@psf.upfronthosting.co.za> |
2010-12-13 09:18:02 | eric.araujo | link | issue9234 messages |
2010-12-13 09:18:01 | eric.araujo | create | |
|