From 74197a9a2dbf59f9c517be85160167372a63ad5e Mon Sep 17 00:00:00 2001 From: =?utf8?q?Daniel=20Mart=C3=AD?= Date: Mon, 4 Jan 2016 21:42:34 +0100 Subject: [PATCH] pep8: start obeying E226 We nearly did already anyway. --- hooks/pre-commit | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/hooks/pre-commit b/hooks/pre-commit index 5872539a..53479051 100755 --- a/hooks/pre-commit +++ b/hooks/pre-commit @@ -46,17 +46,16 @@ else done fi -# In the default configuration, the checks E123, E226 ignored because -# they are not rules unanimously accepted -# On top of those, we ignore: +# We ignore the following PEP8 warnings +# * E123: closing bracket does not match indentation of opening bracket's line +# - Broken if multiple indentation levels start on a single line # * E501: line too long (82 > 79 characters) # - Recommended for readability but not enforced # - Some lines are awkward to wrap around a char limit # * W503: line break before binary operator -# - It's quite new # - Quite pedantic -PEP8_IGNORE="E123,E226,E501,W503" +PEP8_IGNORE="E123,E501,W503" err() { echo ERROR: "$@" -- 2.30.2