Code review comment for ~pelpsi/launchpad:duplicate_ci_jobs

Revision history for this message
Colin Watson (cjwatson) wrote :

Looks like a good start, but I'd definitely be more comfortable with a more detailed test that spelled out the expected output more explicitly, since some of the code doesn't look completely correct.

review: Needs Fixing

« Back to merge proposal