ФорумРазработкаУстановка и администрирование ПОРазное ПО → Ищу готовое решение Mercurial + PHP CodeSniffer

Ищу готовое решение Mercurial + PHP CodeSniffer

  • Абырвалг

    Сообщения: 6476 Репутация: N Группа: Джедаи

    Spritz 5 июля 2011 г. 23:23, спустя 27 секунд

    Сабж. Как бы организовать проверку соблюдения стандартов кодирования?

    Если вешаться на pre-push (pre-receive) hook, насколько я понимаю, проверять нужно только верхние коммиты, которые идут в очереди.

    Может кто-то уже реализовывал - поделитесь, сэкономьте мне время, пожалуйста.
  • mario

    Сообщения: 6067 Репутация: N Группа: Джедаи

    Spritz 5 июля 2011 г. 23:23, спустя 23 часа 59 минут 33 секунды


    Сабж. Как бы организовать проверку соблюдения стандартов кодирования?

    Если вешаться на pre-push (pre-recive) hook, насколько я понимаю, проверять нужно только верхние коммиты, которые идут в очереди.

    Может кто-то уже реализовывал - поделитесь, сэкономьте мне время, пожалуйста.
    с удовольствием посмотрю решение )
  • Абырвалг

    Сообщения: 6476 Репутация: N Группа: Джедаи

    Spritz 5 июля 2011 г. 23:29, спустя 5 минут 54 секунды

    и еще кто как организовывает доступ к репо? Битбакета с его 5-ю пользователями скоро не хватит.

    ну, ssh скорее всего, но может какие-то best practices есть
    Спустя 34 сек.
    а еще можно по крону раз в сутки проверять и слать письма…
    Спустя 69 сек.
    но это наверно не вариант, так как сложно будет понять - кто же именно покорежил оформление
  • mario

    Сообщения: 6067 Репутация: N Группа: Джедаи

    Spritz 5 июля 2011 г. 23:35, спустя 6 минут 55 секунд

    и еще кто как организовывает доступ к репо? Битбакета с его 5-ю пользователями скоро не хватит.

    hg serve тебе в руки ))
    http://adw0rd.ru/2010/mercurial-freebsd/
    Спустя 47 сек.
    по сути можно без центрального репа работать (у каждого поднят свой hg serve), но будет хаос )))
  • Абырвалг

    Сообщения: 6476 Репутация: N Группа: Джедаи

    Spritz 6 июля 2011 г. 0:42, спустя 1 час 6 минут 44 секунды

    да ну, это хуита.

    Должен быть
  • mario

    Сообщения: 6067 Репутация: N Группа: Джедаи

    Spritz 6 июля 2011 г. 2:02, спустя 1 час 19 минут 57 секунд

    да ну, это хуита.

    ну так заюзай https ;)
  • adw0rd

    Сообщения: 22905 Репутация: N Группа: в ухо

    Spritz 6 июля 2011 г. 6:08, спустя 4 часа 6 минут 14 секунд

    юзаю Git + pre-commit hook в котором pep8+pyflakes
    скоро прикручу чекилку документации простую… если руки дойдут… статья будет, ага
    adw/0
  • Абырвалг

    Сообщения: 6476 Репутация: N Группа: Джедаи

    Spritz 6 июля 2011 г. 6:28, спустя 19 минут 39 секунд

    pre-commit hook

    так это ж каждый разраб должен у себя включить
  • adw0rd

    Сообщения: 22905 Репутация: N Группа: в ухо

    Spritz 6 июля 2011 г. 23:12, спустя 16 часов 44 минуты 6 секунд

    Да, именно так, для этого я пробрасываю в репозиторий каталог git-hooks, в котором есть изменяемый pre-commit, а потом делаю симлинк.
    Если контролировать на стороне сервера, в распределенных система контроля версий, то получается что человек уже накомитил N комитов и только после этого хочет слится с основным репозиторием и тут две проблемы:
    1. Надо будет изменять коммиты
    2. Надо будет блокировать репу при каждом пуше от разработчика, пока проверяются все коммиты других разработчиков

    Поэтому разумно каждого разработчика обязать pre-commit хуком
    Спустя 161 сек.
    Правда это часть code review на соответсвие стилю и спецификации, но design review всеравно делать надо. Раньше я ревьювил при слиянии, но потом устал просто, да и других обязаностей набрал… Сейчас мы начинаем юзать Crucible, он прикольный :)
    adw/0
  • Абырвалг

    Сообщения: 6476 Репутация: N Группа: Джедаи

    Spritz 7 июля 2011 г. 0:31, спустя 1 час 18 минут 41 секунду

    а что думаете по поводу проверки по крону раз в сутки? как можно будет отследить, что это именно Вася Пупкин хуево коммитнул?
  • phpdude

    Сообщения: 26624 Репутация: N Группа: в ухо

    Spritz 7 июля 2011 г. 3:33, спустя 3 часа 2 минуты 35 секунд


    а что думаете по поводу проверки по крону раз в сутки? как можно будет отследить, что это именно Вася Пупкин хуево коммитнул?
    вот он пидорас!
    Сапожник без сапог
  • Animator

    Сообщения: 194 Репутация: N Группа: Кто попало

    Spritz 7 июля 2011 г. 4:20, спустя 47 минут 1 секунду

    вчера жена объясняла сыну значение слово "пидорас" :)
    "пидорас" это оказывается "голубой", которого хотят обидеть.
    пхпмайадмин на белорусском нервно курит в коридоре…
  • adw0rd

    Сообщения: 22905 Репутация: N Группа: в ухо

    Spritz 7 июля 2011 г. 13:55, спустя 9 часов 34 минуты 48 секунд


    а что думаете по поводу проверки по крону раз в сутки? как можно будет отследить, что это именно Вася Пупкин хуево коммитнул?
    может тогда комиты тестить, а не весь код в репе?
    Спустя 30 сек.
    тогда будет понятно по комиту кто комитил
    adw/0
  • Абырвалг

    Сообщения: 6476 Репутация: N Группа: Джедаи

    Spritz 7 июля 2011 г. 14:10, спустя 14 минут 45 секунд

    да. Но в коммите будет как я понимаю

    — а
    +++ б


    даже не знаю, корректно ли такое тестить
  • phpdude

    Сообщения: 26624 Репутация: N Группа: в ухо

    Spritz 7 июля 2011 г. 14:15, спустя 4 минуты 48 секунд

    в коммите будет список файлов, которые можно пройтись валидатором и сделать exit 1 если что :-)
    Спустя 190 сек.
    вот почитай. мне понравился подход в свое время, но пока не интегрировал никуда :-)

    http://habrahabr.ru/blogs/php/111977/
    Сапожник без сапог

Пожалуйста, авторизуйтесь, чтобы написать комментарий!