URL: https://github.com/SSSD/sssd/pull/527 Title: #527: PEP8 Fixes and enable PEP8 check in CI
lslebodn commented: """ @mzidek-rh, Could you help me to find a way how to enable catching warning in commit "TESTS: Fix E501 pep8 issues on utils.py"? ``` sh-4.4$ git rev-parse --verify HEAD 47362caf4c30c3ff5a143f4f4a7fd5599db03b48 sh-4.4$ pep8 --version 1.6.2 sh-4.4$ pep8 src/tests/python/sssd/testlib/common/utils.py sh-4.4$ pep8 --max-line-length=79 src/tests/python/sssd/testlib/common/utils.py sh-4.4$ pep8 --max-line-length=79 --select=E501 src/tests/python/sssd/testlib/common/utils.py sh-4.4$ pep8 --ignore=E121,E123,E126,E226,E24,E704,W503,E722 src/tests/python/sssd/testlib/common/utils.py ``` It is also not reproducible with older and newer version (also tested with 1.5.7, 1.7.1 and 2.3.1) Maybe it is reproducible with older version of pep8. But it would be good to see that working on recent systems. BTW patch is correct and it is obvious after checking the file with editor. """ See the full comment at https://github.com/SSSD/sssd/pull/527#issuecomment-370039380
_______________________________________________ sssd-devel mailing list -- sssd-devel@lists.fedorahosted.org To unsubscribe send an email to sssd-devel-le...@lists.fedorahosted.org