From: Daniel Martí <mvdan@mvdan.cc>
Date: Sun, 8 Feb 2015 05:58:24 +0000 (-0800)
Subject: Ignore all the pep8 errors ignored by the default config
X-Git-Tag: 0.4.0~115
X-Git-Url: http://www.chiark.greenend.org.uk/ucgi/~ianmdlvl/git?a=commitdiff_plain;h=9be0d35bd12f7caa2f946d0d46350455c2300285;p=fdroidserver.git

Ignore all the pep8 errors ignored by the default config
---

diff --git a/hooks/pre-commit b/hooks/pre-commit
index c810b5dd..312d57c1 100755
--- a/hooks/pre-commit
+++ b/hooks/pre-commit
@@ -11,6 +11,15 @@ SH_FILES="hooks/pre-commit"
 BASH_FILES="fd-commit jenkins-build docs/update.sh completion/bash-completion"
 RB_FILES="buildserver/cookbooks/*/recipes/*.rb"
 
+# In the default configuration, the checks E123, E133, E226, E241 and E242 are
+# ignored because they are not rules unanimously accepted
+# On top of those, we ignore:
+# * E501: line too long (82 > 79 characters)
+#   - Recommended for readability but not enforced
+#   - Some lines are awkward to wrap around a char limit
+
+PEP8_IGNORE="E123,E133,E226,E241,E242,E501"
+
 err() {
 	echo ERROR: "$@"
 	exit 1
@@ -40,7 +49,7 @@ if ! $PYFLAKES $PY_FILES; then
 	err "pyflakes tests failed!"
 fi
 
-if ! $PEP8 --ignore=E123,E501 $PY_FILES; then
+if ! $PEP8 --ignore=$PEP8_IGNORE $PY_FILES; then
 	err "pep8 tests failed!"
 fi