From 3a64857a4c091f1e5ac71f673ca87ea0580b8bdf Mon Sep 17 00:00:00 2001 From: Tim Abbott Date: Sun, 8 Jan 2017 22:42:19 -0800 Subject: [PATCH] lint: Fix long lines not being treated as actual errors. This led to Travis CI not reporting on line-too-long errors. --- tools/lint-all | 1 + 1 file changed, 1 insertion(+) diff --git a/tools/lint-all b/tools/lint-all index 4418fd46d5..a772890c9f 100755 --- a/tools/lint-all +++ b/tools/lint-all @@ -166,6 +166,7 @@ def build_custom_checkers(by_lang): not re.match("\[[a-z0-9_-]*\]: http.*", line) and "#ignorelongline" not in line and 'migrations' not in fn): print("Line too long (%s) at %s line %s: %s" % (len(line), fn, i+1, line_newline_stripped)) + failed = True lastLine = line if lineFlag and '\n' not in lastLine: print("No newline at the end of file. Fix with `sed -i '$a\\' %s`" % (fn,))