Code review comment for lp:~jelmer/launchpad/600153-qafix

Revision history for this message
Jelmer Vernooij (jelmer) wrote :

Hi Brad,

Thanks for the review.

Since this really is a layering issue (the original code was skipping a layer and looking at the raw data directly, but doing a bad job interpreting that data), I'm not sure if it would make sense to add a test. I don't want to put too much knowledge about how we determine what restricted architecture families are allowed into the publisher tests - that logic is already covered in the archive tests.

What do you think?

« Back to merge proposal