[PATCH 1/3] tox: Remove ignored PEP8 errors and directories

Finucane, Stephen stephen.finucane at intel.com
Tue Dec 22 01:50:24 AEDT 2015


On 10 Dec 20:27, Stephen Finucane wrote:
> Remove the following 'ignore' errors:
> 
>   E121-128,131  various visual indentation errors
>   E251          spaces around keyword / parameter equals
>   H405          multi-line docstring must have summary line
> 
> The only error remaining is E129, which is technically not part of the
> PEP8 standard and contributes in practice besides lots of manual line
> wrapping.
> 
> In addition, there's no good reason why the tests directory shouldn't
> be assessed like everything else - tests are still code and, as such,
> should be held to the same standards..
> 
> Finally, include the 'pwclient' script which should not be ignored.
> 
> Signed-off-by: Stephen Finucane <stephen.finucane at intel.com>

A non-code change that brings around some useful code quality
improvements. Merged.


More information about the Patchwork mailing list