From c44dda988acccfd0450e13b94b9af836700aa65a Mon Sep 17 00:00:00 2001 From: Matheus Marchini Date: Tue, 17 Mar 2020 15:19:11 -0700 Subject: [PATCH] build: fail Travis if commit message check fails "Allowed to fail" tests are easy to miss. With the failure on Travis, new contributors can learn our commit guidelines. Collaborators are still allowed (per our current policy) to land PRs where this check fails, as long as the warnings are fixed before pushing to master. Signed-off-by: Matheus Marchini --- .travis.yml | 1 - 1 file changed, 1 deletion(-) diff --git a/.travis.yml b/.travis.yml index de7d7d47f88681..77694bddf3b05e 100644 --- a/.travis.yml +++ b/.travis.yml @@ -85,4 +85,3 @@ jobs: - flake8 . --count --select=E9,F63,F7,F82 --show-source --statistics allow_failures: # TODO (cclauss): remove this when dependencies are clean - name: "Find syntax errors in our Python dependencies" - - name: "First commit message adheres to guidelines at https://goo.gl/p2fr5Q"