Code review comment for lp:~mbp/launchpad/612171-diff-generation-spam

Revision history for this message
Martin Pool (mbp) wrote :

OK, thanks for your patient reviews, here's another update:

 * add a test that the job wants to retry on pending writes and that no mail is sent
 * fix up the tuple thing
 * simplify (or maybe correct) the code that says the job can be retried

Happily the test does fail if retry_error_types is not a tuple.

« Back to merge proposal