[apparmor] [patch] Update comments in minitools_test.py

Steve Beattie steve at nxnw.org
Mon Jun 1 17:15:33 UTC 2015


On Sun, May 31, 2015 at 07:58:26PM +0200, Christian Boltz wrote:
> Hello,
> 
> After switching to winbindd as test profile, comments about the ntpd
> profile don't make sense anymore ;-)
> 
> The patch also includes a whitespace fix.
> 
> [ 43-update-comments-in-minitools_test.py.diff ]

While you're touching these comments, can you make them pep8 compliant,
by inserting a space after the '#'?

(The specific pep8 issue is "E265 block comment should start with '#
'".  I'm not asking that you address all the pep8 issues in this
script, but since you're touching the comments, we might as well
make them compliant, unless you can make a compelling argument why
we shouldn't).

Acked-by: Steve Beattie <steve at nxnw.org> with that change. Thanks.

-- 
Steve Beattie
<sbeattie at ubuntu.com>
http://NxNW.org/~steve/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20150601/3093ba28/attachment.pgp>


More information about the AppArmor mailing list