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

Revision history for this message
Gavin Panella (allenap) wrote :

Hi Aaron,

I forgot to ping you today about this, so for now this will have to
do. When you have a chance can you have a gander at this, especially
with respect to Jeroen's comment:

> 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.

I'm very bad at doing pre-implementation calls, so I apologise in
advance if I've done something stupid because of the lack of that.

I'm back on Tuesday, so if you'd like to talk about it let's try to
talk then.

Thanks, Gavin.

« Back to merge proposal