Code review comment for lp:~wgrant/launchpad/flatten-bfj-2-garbo

Revision history for this message
Steve Kowalik (stevenk) wrote :

You could *almost* get away with refactoring this into a base class and then having 3 smaller classes that use the base as a super class. But this code should be short lived enough that it probably does not matter.

518 + is_distro_archive = Attribute(
519 + "Whether the target archive belongs to the distro")

Terrible message is terrible.

review: Approve (code)

« Back to merge proposal