From 6cb54e88210cd7fc72ff2479859bb76eb2955a32 Mon Sep 17 00:00:00 2001
From: Hans-Christoph Steiner <hans@eds.org>
Date: Wed, 27 Jun 2018 10:34:04 +0200
Subject: [PATCH] gitlab-ci: name job using all key components:
 test_lint_pmd_checkstyle

---
 .gitlab-ci.yml | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
index f7cad4620..6bcbbb2b9 100644
--- a/.gitlab-ci.yml
+++ b/.gitlab-ci.yml
@@ -30,7 +30,7 @@ before_script:
 
 # Run the most important first. Then we can decide whether to ignore
 # the style tests if the rest of the more meaningful tests pass.
-test:
+test_lint_pmd_checkstyle:
   <<: *test-template
   script:
     - export EXITVALUE=0
@@ -41,8 +41,8 @@ test:
     - ./gradlew lint
     - ./gradlew pmd || export EXITVALUE=1
     - ./gradlew checkstyle || export EXITVALUE=1
-    - ./tools/check-format-strings.py
-    - ./tools/remove-unused-and-blank-translations.py
+    - ./tools/check-format-strings.py || export EXITVALUE=1
+    - ./tools/remove-unused-and-blank-translations.py || export EXITVALUE=1
     - echo "These are unused or blank translations that should be removed:"
     - git --no-pager diff --ignore-all-space --name-only --exit-code app/src/*/res/values*/strings.xml || export EXITVALUE=1
     - exit $EXITVALUE