Pre Commit Hook для JSLint в Mercurial и Git

Я хочу запустить JSLint перед фиксацией в репозитории Mercurial или Git.

Я хочу, чтобы это был автоматический шаг, который был настроен вместо того, чтобы полагаться на разработчика (в основном на меня), не забывая запускать JSLint раньше. Я обычно запускаю JSLint во время разработки, но хочу указать контракт на JS файлах, которые они передают JSLint, прежде чем быть привязанным к репо.

Для Mercurial эта страница описывает синтаксис precommit, но единственными переменными, которые кажутся доступными, являются идентификаторы parent1 и parent2 changeset в фиксации. Я действительно хочу, это список имен файлов, которые связаны с фиксацией, поэтому я могу выбрать файл .js и запустить jslint над ними.

Аналогичная проблема для GIT, информация по умолчанию, доступная как часть precommit script, кажется ограниченной.

Что может быть связано с вызовом статуса hg status/ git как части precommit script, проанализируйте этот вывод, чтобы найти файлы JS, затем выполните эту работу. Я надеялся на что-то более легкое, хотя, и я не уверен, что если статус вызова как часть крючка precommit отражает правильную информацию. Например, в Git, если файлы изменений еще не добавлены, но фиксация Git использует -a, будут ли файлы отображаться в правильном разделе вывода состояния Git как часть набора фиксации?

Обновление: у меня что-то работает, оно видно здесь: http://github.com/jrburke/dvcs_jslint/

Ответ 1

Для git есть примеры в каталоге .git/hooks. Если вам просто нужны имена файлов для JSLint, вы можете использовать git diff --name-only, который в моем примере отобразит имена файлов, которые отличаются от текущего HEAD.

Ответ 2

Ниже приведен вариант решения @Bitbieger Git, который работает с Node.js и локальной копией node-jslint (т.е. вам нужно npm install jslint в каталоге корневого репозитория).

Дополнительно script:

Запускает jslint поверх всех файлов .html и .json, а также .js
Выполняется только запуск jslint над файлами, которые были добавлены, скопированы или изменены. Это предотвращает ошибку jslint для файлов, которые были переименованы или удалены.
Реплицирует любые ошибки jslint для просмотра пользователем
Использует параметры --indent 4 --white true jslint для обеспечения согласованности исходного кода

Чтобы заставить его работать, скопируйте следующее в .git/hooks/pre-commit и не забудьте chmod +x .git/hooks/pre-commit

# Pre-commit hook passing files through jslint
#
# This ensures that all js, html and json files are valid and conform
# to expectations.

ROOT_DIR=$(git rev-parse --show-toplevel)
JSLINT="${ROOT_DIR}/node_modules/.bin/jslint --indent 4 --white true"

for file in $(git diff-index --name-only --diff-filter=ACM --cached HEAD -- | grep -P '\.((js)|(html)|(json))$'); do
    if node $JSLINT $file 2>&1 | grep 'No errors found' ; then
        echo "jslint passed ${file}"
        exit 0
    else
        node $JSLINT $file
        exit 1
    fi  
done

Ответ 3

JSLint с SpiderMonkey

for js in $(git diff-index --name-only --cached HEAD -- | grep '\.js$'); do
    if jslint.sh $js 2>&1 | grep 'Lint at line' ; then
        echo $js
        exit 1
    else
        echo "js files validated"
        exit 0
    fi  
done