Code review comment for lp:~canonical-ca-hackers/ubuntu-webcatalog/1006442-double-distroseries

Revision history for this message
Michael Nelson (michael.nelson) wrote :

14:11 < noodles> lukasz: could lines 31 and 41 do the client.get consistently, or are they doing different things?
14:12 < noodles> lukasz: actually, if they're doing the same thing, why do we need the first test? The one test (the second one) ensures that the status is 200 already?
14:12 < lukasz> noodles, yup, one is for app detail within distroseries, second is for general app detail
14:12 < noodles> lukasz: as, got it. +1

review: Approve

« Back to merge proposal