Launchpad itself

Code review comment for lp:~stevenk/launchpad/subscribers-can-view-p3as

  1. subscribers-can-view-p3as
  2. Merge into devel
Revision history for this message
Robert Collins (lifeless) wrote :

Just a small note on the vertical whitespace mid function: PEP8 says
to use that sparingly to separate logically distinct areas of a single
function; the 'list of things to check' is one big area IMO - I would
delete the vertical white space there.

-Rob

« Back to merge proposal