Landscape Client

Code review comment for lp:~bjornt/landscape-client/apt-facade-reporter-optional

  1. apt-facade-reporter-optional
  2. Merge into trunk
Revision history for this message
Björn Tillenius (bjornt) wrote :

On Wed, Oct 26, 2011 at 07:54:23PM -0000, Mike Milner wrote:
> Good idea +1
>
> There are a few pep8 errors but they seem to be all string literals
> for sample manifests and whatnot. I guess this is a
> know-when-to-bend-the-rules moment :)

Right. Trying to make them to fit would make them less readable
probably.

--
Björn Tillenius | https://launchpad.net/~bjornt

« Back to merge proposal