[Gluster-infra] [Bug 1584992] Need python pep8 and other relevant tests in smoke if a patch includes any python file

bugzilla at redhat.com bugzilla at redhat.com
Fri Jun 8 13:01:28 UTC 2018


https://bugzilla.redhat.com/show_bug.cgi?id=1584992

Aravinda VK <avishwan at redhat.com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |avishwan at redhat.com



--- Comment #3 from Aravinda VK <avishwan at redhat.com> ---
pep8 issues can be fixed by using `autopep8` tool, I will work on it and send
patch.

pylint gives lot of warnings related to variable naming style, we can ignore
those errors. Also pylint errors are mostly about best practices and code
suggestions. I feel we should defer pylint voting since it involves lot of code
changes.

We can use `flake8` for now to validate pep8, unused variables, imports etc.

https://github.com/gluster/libgfapi-python/blob/master/tox.ini

-- 
You are receiving this mail because:
You are on the CC list for the bug.
Unsubscribe from this bug https://bugzilla.redhat.com/token.cgi?t=AWplgKEyLY&a=cc_unsubscribe


More information about the Gluster-infra mailing list