From: Sébastien Han Date: Tue, 3 Jul 2018 08:25:09 +0000 (+0200) Subject: Revert "ceph-ansible-pr-syntax-check: use master instead of HEAD" X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=67fbebc4c20127a9236d4e817ccb876948766d9f;p=ceph-build.git Revert "ceph-ansible-pr-syntax-check: use master instead of HEAD" This reverts commit 7c971b82771ef6766206b7ef0716a8705fdbe649. Because this is the wrong fix, if you push a PR on a stable branch, the comparaison will happen against master, we don't want that. --- diff --git a/ceph-ansible-pr-syntax-check/build/build b/ceph-ansible-pr-syntax-check/build/build index 199e0d82c..e5b3301c7 100644 --- a/ceph-ansible-pr-syntax-check/build/build +++ b/ceph-ansible-pr-syntax-check/build/build @@ -47,7 +47,7 @@ function group_vars_check { } function test_sign_off { - test "$(git log --oneline origin/master..HEAD | wc -l)" -ne "$(git log origin/master..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 origin/HEAD..HEAD | wc -l)" -ne "$(git log origin/HEAD..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 }