Code review comment for lp:~rvb/maas/zone-listing-pagination

Revision history for this message
Raphaƫl Badin (rvb) wrote :

> I see you applied the comments from my other review here as well. That's why
> I put this one off for a while. :) Thanks.
>
> The "Create 4 zones" comment probably makes a bit more sense here than the
> "Create 4 networks" one in the other branch. But when you add it to the two-
> line list comprehension, it's not much nicer than the loop it replaces...
> Maybe just put the list comprehension on a single line?

Right, done.

Thanks for the review!

« Back to merge proposal