Как стать автором
Обновить

Комментарии 48

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

Автору: Таки поправьте наконец! В заголовке необходимо указать, что речь идёт только о форматировании кода, а не проверке вообще. Сейчас заголовок вводит в заблуждение.
Не знаю — на меркуриал ставится все на ура, после траты 20 минут на ман.
1. А на какой обработчкик вы его к HG прикрутили?
2. Каждому раз работчику или в интеграционный репозиторий (да я понимаю, что у вас может быть другой workflow)?
1. pretxncommit
2. каждому
НЛО прилетело и опубликовало эту надпись здесь
Всё хорошо и правильные, но русские комментарии — моветон.
НЛО прилетело и опубликовало эту надпись здесь
НЛО прилетело и опубликовало эту надпись здесь
Если честно, я поражен вашим вопросом. Вы же не пишете код на русском?
Но и на английском я его не пишу. Я его пишу на языке программирования.
Но программист из любой другой страны ваш код поймет, а комментарии — никогда.
НЛО прилетело и опубликовало эту надпись здесь
По этому поводу чуть выше заметили — если ваша система для внутреннего использования, то не имеет значения (я сам всегда на английском пишу комментарии если что)
Потому что вы ограничиваете множество людей, которые будут читать ваш код до русскоговорящих. Разве не очевидно?
НЛО прилетело и опубликовало эту надпись здесь
Это просто стиль кодирования, стандарт. В конце концов в русском языке как правило нет подходящей IT-терминологии, названия методов и переменных тоже на английском, в итоге вы получаете полную кашу.
Любой специалист прочтет ваш комментарий, если нет — то он отвратительный специалист и не место ему в сфере.

Вы завтра пойдете устраиваться на работу в иностранную компанию, и покажете свой лучший исходник, а там бац, и комментарии на русском языке.
НЛО прилетело и опубликовало эту надпись здесь
Стили и стандарты кодирования регламентируются, как правило, внутренними договоренностями команды. Если команда не планирует расширяться за счёт нерусскоговорящих, то комментарии на русском вполне допустимы, имхо, а если команда ещё и английским владеет не свободно (как я пишу в резюме «читаю документацию на ПО, иногда со словарём»), то даже желательны — я могу, наверное, так что-нибудь написать, что человек хорошо знающий английский или вообще не поймёт, или поймёт неправильно, если не учился в советской школе :) А другой «полиглот» вставит какую-нибудь идиому, поговорку или цитату, над которой вся команда полдня будет думать, пытаясь перевести дословно
НЛО прилетело и опубликовало эту надпись здесь
Если уж пишете на русском, пишите на нём правильно: «русский», «английский», всё с маленькой буквы.
НЛО прилетело и опубликовало эту надпись здесь
PHPUnit + Code Sniffer + PMD незаменимы в большом (и не очень) PHP-проекте в связке с системой непрерывной интеграции. А если ещё и работаем по Scrum — сказка
pmd — это в java. в php — phpmd ;)
Если не сложно, расскажите про систему непрерывной интеграции — что это такое, как настроить, и что дает.
Дичайший язык изложения, особенно с деепричастными оборотами, — сильно похоже на корявый перевод. И ни одного комментария автора.

А по теме: рекомендую еще статью Бергманна Quality Assurance Tools For PHP — там описана целая система инструментов, включая и phpcs.
Приношу извинения автору: прочитал другие статьи — стиль, вроде, тот же самый :) Ошибки есть, что не уменьшает пользы самих статей.
А можно как-то настроить чтобы проверялся только добавленый/измененный код?
Так хук и должен проверять только файлы в коммите.
У git'a в коммит попадают только новые/изменённые файлы.
Это да, но я не хочу проверять весь файл. Мне надо только кусок.
Скажем так: есть файл на 10 тыс. строк, его рефакторят. Но за 1 раз отрефакторить все невозможно!
Теперь понял.
<?php
много кода
// @codingStandardsIgnoreStart
здесь тоже много кода, но он не будет проверен
// @codingStandardsIgnoreEnd
дальше много кода

Вот, то что нужно. Круто. Спасибо!
собираемся у себя в проекте использовать эту систему, пока только 2 разработчика, но по идее чем раньше начнешь, тем чище будет код =)
Лучше бы он, сука, русский язык проверял. Извините.
Стандарт Zend что в комплекте не стандарт соответствует стандарту описанному на офф. сайте, обещали что-то сделать к 2.0 в этом направлении.
Ах да еще можно php -l воткнуть чтобы синтаксис проверялся.
У кого-нибудь получилось установить pre-commit хук под windows?
Спасибо, бро. у меня тогда вроде тоже всё получилось в итоге.
Как долго я набирал свой Standard для этого зверя! Не нашёл в природе ни одного конструктора, хотя оный был бы полезен многим.
Вы конструктор так и не нашли после коммента?
И не искал. Не возникло необходимости, так как вся работа и так была проведена.
Можно при сохранении файла проверять. Что бы разработчик сразу делал правки, а не один раз при коммите, перебирая все файлы.
Плюс, проверка при сохранении, как по мне, приучает писать код правильнее.
Зарегистрируйтесь на Хабре , чтобы оставить комментарий

Публикации

Истории