On (01/09/15 14:37), Pavel Reichl wrote: >This patch fixes some of the pep8 warnings (number of issues reported by pep8 >utility changed from 204 to 127). >Should we file a ticket to reduce pep8 warning as much as possible? I suppose >we could even add a test that would fail if new pep8 warning is introduced, >but I'm little worried that it might prove to be too difficult to write such >a test in quality that it will be accepted by all SSSD developers. What do >you think? > The quite recent test src/tests/intg/test_memory_cache.py does not have any pep8 warnings or errors. However there is old python code with many warnings which would prevent us from wrinting such test.
But there is still way how to test pep8 warnings in new python code. sh$ pep8 --help //snip --diff report only lines changed according to the unified diff received on STDIN If we fix/suppress all pep8 warnings then we can write test. LS _______________________________________________ sssd-devel mailing list sssd-devel@lists.fedorahosted.org https://lists.fedorahosted.org/mailman/listinfo/sssd-devel