Как предотвратить git commit, если определенная фраза в изменениях обнаружена?

Я заметил некоторые паттерны, к которым склонны разработчики — например, коммит тестов javascript с оставленными в них fdescribe или fit (что означает, что будет запущен только один тест/набор), обычно это обнаруживается на этапе проверки, но было бы неплохо поймать эти мелочи ранее. Хотите знать, есть ли способ настроить git для предотвращения фиксации, если в изменениях обнаружен какой-то шаблон?


git atlassian-sourcetree bitbucket-server
person Giedrius    schedule 30.09.2015    source источник
comment
Хук pre-commit может делать такие вещи, чтобы направить вас на правильный путь   -  person Tim    schedule 30.09.2015
comment
Я бы рекомендовал делать это не перед коммитом, а перед push-хуком; git поощряет разработчиков часто делать коммиты, но они не должны отправлять все в общий репозиторий.   -  person Marcus Müller    schedule 30.09.2015

Ответы (1)


arrow_upward
5
arrow_downward

Это классическая работа для git commit hooks (man githooks); например, из стандартных примеров скриптов, которые вы получаете в .git/hooks, вы можете добавить скрипт pre-push в эту папку (сделать его исполняемым!).

Этот пример скрипта из проекта git проверяет только имена коммитов, но если вы замените имя

git rev-list

линия с чем-то вроде

git diff $remote_sha $local_sha

и grep для ваших подозрительных строк, например.

git diff $remote_sha $local_sha|grep -E '^\+.*(fdescribe|fit)'

вы можете сделать это для вашего случая.

#!/bin/sh

# An example hook script to verify what is about to be pushed.  Called by "git
# push" after it has checked the remote status, but before anything has been
# pushed.  If this script exits with a non-zero status nothing will be pushed.
#
# This hook is called with the following parameters:
#
# $1 -- Name of the remote to which the push is being done
# $2 -- URL to which the push is being done
#
# If pushing without using a named remote those arguments will be equal.
#
# Information about the commits which are being pushed is supplied as lines to
# the standard input in the form:
#
#   <local ref> <local sha1> <remote ref> <remote sha1>
#
# This sample shows how to prevent push of commits where the log message starts
# with "WIP" (work in progress).

remote="$1"
url="$2"

z40=0000000000000000000000000000000000000000

while read local_ref local_sha remote_ref remote_sha
do
    if [ "$local_sha" = $z40 ]
    then
        # Handle delete
        :
    else
        if [ "$remote_sha" = $z40 ]
        then
            # New branch, examine all commits
            range="$local_sha"
        else
            # Update to existing branch, examine new commits
            range="$remote_sha..$local_sha"
        fi

        # Check for WIP commit
        commit=`git rev-list -n 1 --grep '^WIP' "$range"`
        if [ -n "$commit" ]
        then
            echo >&2 "Found WIP commit in $local_ref, not pushing"
            exit 1
        fi
    fi
done

exit 0
person Marcus Müller    schedule 30.09.2015
comment
Это не работает при нажатии новой ветки: git diff $local_sha ничего не возвращает - person ricab; 31.01.2019