CI: Enforce checkcommits.

checkcommits has been running successfuly for a while now, so disallow it failing in a CI environment. Additionally, only run it in a TravisCI environment since it cannot currently work under SemaphoreCI (due to its restrictive environment). Fixes #967 (but actually fixed by clearcontainers/tests#64). Signed-off-by: James O. D. Hunt <[email protected]>
intel · Jun 23, 2017 · 55eba02 · 55eba02
1 parent f92d50a
commit 55eba02
Showing 1 changed file with 12 additions and 9 deletions.
diff --git a/.ci/ci-pre-checks.sh b/.ci/ci-pre-checks.sh
@@ -23,12 +23,15 @@
 #   will be aborted.
 #---------------------------------------------------------------------
 
-repo="github.com/clearcontainers/tests/cmd/checkcommits"
-go get -d "$repo"
-(cd "$GOPATH/src/$repo" && make)
-checkcommits \
-    --need-fixes \
-    --need-sign-offs \
-    --body-length 72 \
-    --subject-length 75 \
-    --verbose || true
+if [ "$TRAVIS" = true ]
+then
+    repo="github.com/clearcontainers/tests/cmd/checkcommits"
+    go get -d "$repo"
+    (cd "$GOPATH/src/$repo" && make)
+    checkcommits \
+        --need-fixes \
+        --need-sign-offs \
+        --body-length 72 \
+        --subject-length 75 \
+        --verbose
+fi