Code review comment for lp:~stevenk/launchpad/notify-announce-list

Revision history for this message
William Grant (wgrant) wrote :

133 + if (action == 'accepted' and distroseries.changeslist and
134 + not archive.is_ppa and pocket != PackagePublishingPocket.BACKPORTS
135 + and not (pocket == PackagePublishingPocket.SECURITY and spr is None)
136 + and not is_auto_sync_upload(
137 + spr, bprs, pocket, changes['Changed-By'])):

The ands were consistently at the end of each line. Either is OK, but not both at once please.

review: Approve

« Back to merge proposal