[Freeipa-devel] [PATCH 0379 - 0393] Winter code cleanup

Martin Basti mbasti at redhat.com
Thu Dec 17 01:11:40 UTC 2015


Hello,
following patches cleanup the code and add checks to pylint to avoid the 
mess again

Patches: 379-381:
the most important patches, they cleanup imports

Patch 382:
enables various pylint checks, we may reduce the list of check if needed.

Patches 383 - 393:
remove minor issues from code, and enable particular checks
Feel free to nack patches/checks that should not be there.

Please apply patches in order.

Martin^2
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0379-Remove-empty-test-file.patch
Type: text/x-patch
Size: 1537 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0380-Remove-unused-imports.patch
Type: text/x-patch
Size: 104888 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0001.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0381-Remove-wildcard-imports.patch
Type: text/x-patch
Size: 44245 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0002.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0382-Enable-multiple-warnings-checks-in-Pylint.patch
Type: text/x-patch
Size: 2386 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0003.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0383-Enable-pylint-lost-exception-check.patch
Type: text/x-patch
Size: 1685 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0004.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0384-Enable-pylint-duplicated-key-check.patch
Type: text/x-patch
Size: 1954 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0005.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0385-Enable-pylint-trailing-whitespace-check.patch
Type: text/x-patch
Size: 3905 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0006.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0386-Enable-pylint-missing-final-newline-check.patch
Type: text/x-patch
Size: 1664 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0007.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0387-Enable-pylint-unused-format-string-key-check.patch
Type: text/x-patch
Size: 2760 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0008.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0388-Enable-pylint-expression-not-assigned-check.patch
Type: text/x-patch
Size: 8097 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0009.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0389-Enable-pylint-pointless-statement-check.patch
Type: text/x-patch
Size: 5280 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0010.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0390-Enable-pylint-empty-docstring-check.patch
Type: text/x-patch
Size: 2630 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0011.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0391-Enable-pylint-unnecessary-lambda-check.patch
Type: text/x-patch
Size: 5916 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0012.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0392-Enable-pylint-eval-and-exec-usage-checks.patch
Type: text/x-patch
Size: 2227 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0013.bin>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: freeipa-mbasti-0393-Enable-pylint-unnecessary-pass-check.patch
Type: text/x-patch
Size: 11013 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/freeipa-devel/attachments/20151217/e0a2c2ea/attachment-0014.bin>


More information about the Freeipa-devel mailing list