From 55eba02e6602be11a14c4b229193c4679851a81b Mon Sep 17 00:00:00 2001 From: "James O. D. Hunt" Date: Fri, 23 Jun 2017 14:55:17 +0100 Subject: [PATCH] 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 https://github.com/clearcontainers/tests/pull/64). Signed-off-by: James O. D. Hunt --- .ci/ci-pre-checks.sh | 21 ++++++++++++--------- 1 file changed, 12 insertions(+), 9 deletions(-) diff --git a/.ci/ci-pre-checks.sh b/.ci/ci-pre-checks.sh index e3c8418e..fa479f05 100755 --- 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