Launchpad itself

Code review comment for lp:~al-maisan/launchpad/oops-526969

  1. oops-526969
  2. Merge into devel
Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

Looks fine, and the TAL ends up cleaner as well. The new tests check for the effect of the Job's status; the other half of the check is covered by existing tests.

Land it.

review: Approve (code)

« Back to merge proposal