Code review comment for lp://staging/~adeuring/ci-director/archive-build-summary

Revision history for this message
Aaron Bentley (abentley) wrote :

This looks good. Please rename send_final_mail to something else, now that it is doing more. Maybe finish() or finalize() or wrap_up()?

review: Approve

« Back to merge proposal