From: Daniel Martí <mvdan@mvdan.cc>
Date: Mon, 4 Jan 2016 20:42:34 +0000 (+0100)
Subject: pep8: start obeying E226
X-Git-Tag: 0.7.0~86^2~10
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=74197a9a2dbf59f9c517be85160167372a63ad5e;p=fdroidserver.git

pep8: start obeying E226

We nearly did already anyway.
---

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: "$@"