[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