diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml index e35362717..224d407cb 100644 --- a/.gitlab-ci.yml +++ b/.gitlab-ci.yml @@ -1,6 +1,6 @@ include: - - remote: 'https://gitlab.freedesktop.org/freedesktop/ci-templates/-/raw/8445ff7af2a68795afb98f486251f2ef8f90621c/templates/fedora.yml' - - remote: "https://gitlab.freedesktop.org/freedesktop/ci-templates/-/raw/8445ff7af2a68795afb98f486251f2ef8f90621c/templates/ci-fairy.yml" + - remote: 'https://gitlab.freedesktop.org/freedesktop/ci-templates/-/raw/18194044f0f984c8815bc9a1a146582f6bf15d41/templates/fedora.yml' + - remote: "https://gitlab.freedesktop.org/freedesktop/ci-templates/-/raw/6f86b8bcb0cd5168c32779c4fea9a893c4a0c046/templates/ci-fairy.yml" stages: - review @@ -85,7 +85,12 @@ check-commit-log: variables: GIT_DEPTH: "100" script: - - ./.gitlab-ci/check-commit-log.sh + - if [[ x"$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" != "x" ]] ; + then + ci-fairy check-commits --junit-xml=commit-message-junit-report.xml ; + else + echo "Not a merge request" ; + fi artifacts: expire_in: 1 week paths: diff --git a/.gitlab-ci/check-commit-log.sh b/.gitlab-ci/check-commit-log.sh deleted file mode 100755 index d8b65c195..000000000 --- a/.gitlab-ci/check-commit-log.sh +++ /dev/null @@ -1,121 +0,0 @@ -#!/usr/bin/env bash - -if [ -z "$CI_MERGE_REQUEST_TARGET_BRANCH_NAME" ]; then - echo This is not a merge request, skipping - exit 0 -fi - -git fetch $CI_MERGE_REQUEST_PROJECT_URL.git $CI_MERGE_REQUEST_TARGET_BRANCH_NAME - -branch_point=$(git merge-base HEAD FETCH_HEAD) - -commits=$(git log --format='format:%H' $branch_point..$CI_COMMIT_SHA) - -if [ -z "$commits" ]; then - echo Commit range empty - exit 1 -fi - -function commit_message_has_mr_url() { - commit=$1 - commit_message=$(git show -s --format='format:%b' $commit) - echo "$commit_message" | grep -qe "^$CI_MERGE_REQUEST_PROJECT_URL\/\(-\/\)\?merge_requests\/$CI_MERGE_REQUEST_IID$" - return $? -} - -JUNIT_REPORT_TESTS_FILE=$(mktemp) - -function append_failed_test_case() { - test_name="$1" - commit="$2" - test_message="$3" - commit_short=${commit:0:8} - - echo "" >> $JUNIT_REPORT_TESTS_FILE - echo >&2 "Commit check failed: $commit_short: $test_message" -} - -function append_passed_test_case() { - test_name="$1" - commit="$2" - commit_short=${commit:0:8} - - echo "" >> $JUNIT_REPORT_TESTS_FILE -} - -function generate_junit_report() { - junit_report_file="$1" - num_tests=$(cat "$JUNIT_REPORT_TESTS_FILE" | wc -l) - num_failures=$(grep ' $junit_report_file << __EOF__ - - - -$(< $JUNIT_REPORT_TESTS_FILE) - - -__EOF__ -} - -function check_commit_message_subject() { - commit=$1 - commit_message_subject=$(git show -s --format='format:%s' $commit) - - if echo "$commit_message_subject" | grep -qe "\(^meta-\|^Meta\)"; then - append_failed_test_case meta-prefix $commit \ - "Commit message subject should not be prefixed with 'meta-' or 'Meta'" - else - append_passed_test_case meta-prefix $commit - fi - - if echo "$commit_message_subject" | grep -qe "\(^clutter-\|^Clutter\)"; then - append_failed_test_case clutter-prefix $commit \ - "Commit message subject should not be prefixed with 'clutter-' or 'Clutter', use 'clutter/' instead" - else - append_passed_test_case clutter-prefix $commit - fi - - if echo "$commit_message_subject" | grep -qe "\(^cogl-\|^Cogl\)"; then - append_failed_test_case cogl-prefix $commit \ - "Commit message subject should not be prefixed with 'cogl-' or 'Cogl', use 'cogl/' instead" - else - append_passed_test_case cogl-prefix $commit - fi - - if echo "$commit_message_subject" | sed -e 's/^[^:]\+: //' | grep -qe '^[[:lower:]]'; then - append_failed_test_case capitalization $commit \ - "Commit message subject should be properly Capitalized. E.g. 'window: Marginalize extradicity'" - else - append_passed_test_case capitalization $commit - fi - - if echo "$commit_message_subject" | grep -qe "\.[ch]:"; then - append_failed_test_case not-file-suffix $commit \ - "Commit message subject prefix should not include .c, .h, etc." - else - append_passed_test_case not-file-suffix $commit - fi - - return 0 -} - -RET=0 -for commit in $commits; do - - if commit_message_has_mr_url $commit; then - append_failed_test_case superfluous_url $commit \ - "Commit message must not contain a link to its own merge request" - else - append_passed_test_case superfluous_url $commit - fi - - check_commit_message_subject $commit -done - -generate_junit_report commit-message-junit-report.xml - -! grep -q '