Как настроить git на проверку var_dump в коде перед коммитом?

Question

[Александр Арбузов]

Как настроить git на проверку кода перед коммитом, что бы в коде не было отладочных вещей всяких, var_dump, console.log и т.п.?

Answer 1

[l4m3r]

pre-commit hook?

Comments

[Александр Арбузов]

Спасибо. Я знаю что такое есть, но не в курсе был, как именно делается.

Answer 2

[Александр Новиков]

1) Подключить phpcs для проверки code style (конкретно настроить https://github.com/squizlabs/PHP_CodeSniffer/wiki/... секция Generic.PHP.ForbiddenFunctions)
2) Автоматизировать проверку на стороне гита - precommit hook (например https://habr.com/post/111977/ тут пример)
3) В идеале если разработчик не один - то в процесс CI тоже добавить запуск этой таски phpcs (например, в Jenkins).

Так как локально кто-то может и не настроить хук, а удаленный билд хочешь-не хочешь, а проверит.

Т.е оптимальный вариант такой
1) добавить phpcs в composer
2) написать нужный ruleset (в вашем случае - с указанием genericphpforbiddenfunctions)
3) добавить в сборщик (напимер phing) таску на запуск phpcs с конфигами нужного ruleset
4) добавить локально хук на запуск phing phpcs
5) добавить запуск phing phpcs в билд дженкинса

Answer 3

[Евгений Комаров]

1. Забыть про использование "отладочных вещей всяких, var_dump, console.log и т.п." (использовать нормальные debug-инструменты)
2. Core-review перед пушем в прод

Comments

[Александр Арбузов]

Ваше мнение о методах разработки бесконечно ценно для меня. Какой из предложенных вариантов отвечает на мой вопрос?

[l4m3r]

. Забыть про использование "отладочных вещей всяких, var_dump, console.log и т.п." (использовать нормальные debug-инструменты)

Например?

[Сергей Соколов]

может не "coRe-review" а "code review" ?

[Mysterion]

l4m3r, зачем советовать отказаться от использования таких вещей, если они нужны автору, при этом имея возможность предложить реальное решение, которое не заставляет отказываться от использования этих функций?

[l4m3r]

Mysterion, вы точно мне?

[Mysterion]

l4m3r, в том числе.

[l4m3r]

Mysterion, ну да, незачем. Я и не спорил.

[Дмитрий]

Евгений Комаров, просветите чайников списком нормальных debug-инструментов!