Автоматизированная проверка PHP кода при комитах

[DevMan]

Мы используем pre-update на мастере вместо pre-commit у каждого разработчика.
Стиль форматирования и прочие плюшки вида skipsniff задаётся через git config.

[DevMan]

И поправьте название — не проверка кода, а проверка стиля форматирования кода.

[rubyrabbit]

Жаль, что автор вас не слышит.

Автору: Таки поправьте наконец! В заголовке необходимо указать, что речь идёт только о форматировании кода, а не проверке вообще. Сейчас заголовок вводит в заблуждение.

[SowingSadness]

Не знаю — на меркуриал ставится все на ура, после траты 20 минут на ман.

[durach]

1. А на какой обработчкик вы его к HG прикрутили?
2. Каждому раз работчику или в интеграционный репозиторий (да я понимаю, что у вас может быть другой workflow)?

[SowingSadness]

1. pretxncommit
2. каждому

[Aivean]

Всё хорошо и правильные, но русские комментарии — моветон.

[theRavel]

Если честно, я поражен вашим вопросом. Вы же не пишете код на русском?

[seriyPS]

Но и на английском я его не пишу. Я его пишу на языке программирования.

[theRavel]

Но программист из любой другой страны ваш код поймет, а комментарии — никогда.

[seriyPS]

По этому поводу чуть выше заметили — если ваша система для внутреннего использования, то не имеет значения (я сам всегда на английском пишу комментарии если что)

[Aivean]

Потому что вы ограничиваете множество людей, которые будут читать ваш код до русскоговорящих. Разве не очевидно?

[theRavel]

Это просто стиль кодирования, стандарт. В конце концов в русском языке как правило нет подходящей IT-терминологии, названия методов и переменных тоже на английском, в итоге вы получаете полную кашу.
Любой специалист прочтет ваш комментарий, если нет — то он отвратительный специалист и не место ему в сфере.

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

[VolCh]

Стили и стандарты кодирования регламентируются, как правило, внутренними договоренностями команды. Если команда не планирует расширяться за счёт нерусскоговорящих, то комментарии на русском вполне допустимы, имхо, а если команда ещё и английским владеет не свободно (как я пишу в резюме «читаю документацию на ПО, иногда со словарём»), то даже желательны — я могу, наверное, так что-нибудь написать, что человек хорошо знающий английский или вообще не поймёт, или поймёт неправильно, если не учился в советской школе :) А другой «полиглот» вставит какую-нибудь идиому, поговорку или цитату, над которой вся команда полдня будет думать, пытаясь перевести дословно

[bolk]

Если уж пишете на русском, пишите на нём правильно: «русский», «английский», всё с маленькой буквы.

[pro100tak]

PHPUnit + Code Sniffer + PMD незаменимы в большом (и не очень) PHP-проекте в связке с системой непрерывной интеграции. А если ещё и работаем по Scrum — сказка

[DevMan]

pmd — это в java. в php — phpmd ;)

[romka777]

Если не сложно, расскажите про систему непрерывной интеграции — что это такое, как настроить, и что дает.

[DevMan]

что это такое и что дает
как настроить

[lalaki]

Дичайший язык изложения, особенно с деепричастными оборотами, — сильно похоже на корявый перевод. И ни одного комментария автора.

А по теме: рекомендую еще статью Бергманна Quality Assurance Tools For PHP — там описана целая система инструментов, включая и phpcs.

[lalaki]

Приношу извинения автору: прочитал другие статьи — стиль, вроде, тот же самый :) Ошибки есть, что не уменьшает пользы самих статей.

[AndrewStephanoff]

кстати, для Eclipse есть замечательный плагин www.phpsrc.org/

[Nerten]

А можно как-то настроить чтобы проверялся только добавленый/измененный код?

[DevMan]

Так хук и должен проверять только файлы в коммите.

[DevMan]

У git'a в коммит попадают только новые/изменённые файлы.

[Nerten]

Это да, но я не хочу проверять весь файл. Мне надо только кусок.
Скажем так: есть файл на 10 тыс. строк, его рефакторят. Но за 1 раз отрефакторить все невозможно!

[DevMan]

Теперь понял.

<?php
много кода
// @codingStandardsIgnoreStart
здесь тоже много кода, но он не будет проверен
// @codingStandardsIgnoreEnd
дальше много кода

[Nerten]

Вот, то что нужно. Круто. Спасибо!

[kiote]

собираемся у себя в проекте использовать эту систему, пока только 2 разработчика, но по идее чем раньше начнешь, тем чище будет код =)

[redhummer]

Лучше бы он, сука, русский язык проверял. Извините.

[mecommayou]

Стандарт Zend что в комплекте не стандарт соответствует стандарту описанному на офф. сайте, обещали что-то сделать к 2.0 в этом направлении.

[mecommayou]

Ах да еще можно php -l воткнуть чтобы синтаксис проверялся.

[Sonic_SE]

У кого-нибудь получилось установить pre-commit хук под windows?

[plutov]

Да, у меня получилось — plutov.by/post/git_pre_commit_windows

[Sonic_SE]

Спасибо, бро. у меня тогда вроде тоже всё получилось в итоге.

[huze]

Как долго я набирал свой Standard для этого зверя! Не нашёл в природе ни одного конструктора, хотя оный был бы полезен многим.

[DenisOgr]

Вы конструктор так и не нашли после коммента?

[huze]

И не искал. Не возникло необходимости, так как вся работа и так была проведена.

[DenisOgr]

Можно при сохранении файла проверять. Что бы разработчик сразу делал правки, а не один раз при коммите, перебирая все файлы.
Плюс, проверка при сохранении, как по мне, приучает писать код правильнее.

[plutov]

Если кому-то инттересно, то я в своём блоге описал решение для Windows.