]> git.apps.os.sepia.ceph.com Git - ceph-build.git/commitdiff
ceph-ansible-pr-syntax-check: remove merge commit from count 1068/head
authorSébastien Han <seb@redhat.com>
Tue, 3 Jul 2018 10:18:30 +0000 (12:18 +0200)
committerSébastien Han <seb@redhat.com>
Tue, 3 Jul 2018 10:29:23 +0000 (12:29 +0200)
When counting the number of commit you must remove the merge commit so
we get the right number of commits.

Signed-off-by: Sébastien Han <seb@redhat.com>
ceph-ansible-pr-syntax-check/build/build

index 2be7439a85640515878dfe45d17dfbb9aba035e7..4f1e71eebc51716dcc37e8c3e779ab91a529dd69 100644 (file)
@@ -50,7 +50,7 @@ function group_vars_check {
 function test_sign_off {
   git fetch
   # shellcheck disable=SC2154
-  test "$(git log --oneline origin/"${ghprbTargetBranch}"..HEAD | wc -l)" -ne "$(git log origin/"${ghprbTargetBranch}"..HEAD | grep -c Signed-off-by)" && echo "One or more commits is/are missing a Signed-off-by. Add it with 'git commit -s'."
+  test "$(git log --oneline --no-merges origin/"${ghprbTargetBranch}"..HEAD | wc -l)" -ne "$(git log --no-merges origin/"${ghprbTargetBranch}"..HEAD | grep -c Signed-off-by)" && echo "One or more commits is/are missing a Signed-off-by. Add it with 'git commit -s'."
   return 1
 }