Code review comment for lp:~allenap/launchpad/job-status-validator

Revision history for this message
Jeroen T. Vermeulen (jtv) wrote :

This one's bitten me too, so I'll be very glad to see it go.

One thing I missed was a pre-imp. Gavin, did you verify that there isn't some good but invisible reason for the current design? I believe Aaron wrote this, and he seems to prefer explaining his code in person over leaving comments.

review: Needs Fixing (code)

« Back to merge proposal