ФорумСообществоПрофессиональная деятельностьВзаимопомощь → Нужно решение для ревью кода

Нужно решение для ревью кода

  • Абырвалг

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

    Spritz 20 сентября 2011 г. 10:02

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

    Гитхаб не подходит тем, что там гит, и бесплатно он такого не предоставляет. Что-то типа redmine + code review плагин на нее, только это ж нужно устанавливать и настраивать редмайн, а у меня все времени нет.
  • phpdude

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

    Spritz 20 сентября 2011 г. 10:15, спустя 13 минут 4 секунды

    а у меня все времени нет.

    ребетишек запарь)
    Сапожник без сапог
  • Ivan

    Сообщения: 1316 Репутация: N Группа: Адекваты

    Spritz 20 сентября 2011 г. 10:20, спустя 4 минуты 39 секунд

    Гитхаб не подходит тем, что там гит

    http://hg-git.github.com/
  • Абырвалг

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

    Spritz 20 сентября 2011 г. 11:36, спустя 1 час 16 минут 54 секунды

    ребетишек запарь)

    да они не шарят такое. Набокопорят еще на серваке

    http://hg-git.github.com/

    та не, это дрюч туда-сюда гонять, +платить же ж еще нужно, у нас около 8 приватных репо
  • disc

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

    Spritz 20 сентября 2011 г. 11:47, спустя 10 минут 58 секунд

    Абырвалг, http://www.atlassian.com/software/crucible/ в связке с jira оно работает, но может отдельно также куда то прикрутить получится.
  • Абырвалг

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

    Spritz 20 сентября 2011 г. 11:56, спустя 8 минут 50 секунд

    да, продукты атлассиана крутые. Но боюсь, что это установить будет куда сложнее редмайна.

    короче чувствую, что редмайн все же буду ставить, но вы все равно кидайте всякие тулзы/сервисы, интересно что есть в мире
    Спустя 47 сек.
    что интересно - вот битбакет от атлассиана тоже же. Но там комментарии чисто к коммиту в целом можно оставлять, но не к конкретной строке
  • disc

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

    Spritz 20 сентября 2011 г. 12:28, спустя 31 минуту 26 секунд


    что интересно - вот битбакет от атлассиана тоже же. Но там комментарии чисто к коммиту в целом можно оставлять, но не к конкретной строке

    а ты в комменте строку указывай и кусок кода
  • adw0rd

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

    Spritz 21 сентября 2011 г. 11:22, спустя 22 часа 54 минуты 18 секунд


    Абырвалг, http://www.atlassian.com/software/crucible/ в связке с jira оно работает, но может отдельно также куда то прикрутить получится.
    +1, сейчас его начинаем юзать, удобный
    + pre-commit хуки для предварительного Code Review, проверка стилю, проверка формата комментариев и т.д., короче все что можно автоматизизировать
    adw/0
  • Абырвалг

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

    Spritz 21 сентября 2011 г. 12:28, спустя 1 час 6 минут 21 секунду

    я наверно буду ставить крякнутую версию

    + pre-commit хуки для предварительного Code Review, проверка стилю, проверка формата комментариев и т.д., короче все что можно автоматизизировать

    расскажи пожалуйста как это работает в связке с гитом (вы ж его используете?)? У каждого разработчика хук стоит что ли?
  • Абырвалг

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

    Spritz 21 сентября 2011 г. 13:34, спустя 1 час 5 минут 43 секунды

    This is why we are excited to announce that Atlassian has donated $24,000 USD to sponsor Mercurial development for 2012.

    а гиту ни копейки не дали, вухаха, так ему и нада!
  • adw0rd

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

    Spritz 21 сентября 2011 г. 16:25, спустя 2 часа 51 минуту 6 секунд


    + pre-commit хуки для предварительного Code Review, проверка стилю, проверка формата комментариев и т.д., короче все что можно автоматизизировать

    расскажи пожалуйста как это работает в связке с гитом (вы ж его используете?)? У каждого разработчика хук стоит что ли?


    Определены два хука для Git, pre-commit и commit-msg
    Сейчас в pre-commit проверяется pep8, pyflakes, почти докрутил проверку докстрингов, можно прикрутить unittests и т.д.
    А в commit-msg проверяется формат сообщения, примерно такой вид требуется "Тема задачи в jira [Идентификатор задачи в JIra] Описание того что было проделано в этом коммите"
    Спустя 105 сек.
    Так же у нас на каждую задачу создается по бранчу (90-95% покрытие бранчами задач), поэтому существует требование к именованию бранча "типЗадачи_идентификаторЗадачиВJira_кратноеОписаниеЗадачиЧащеТемаИзJira"
    adw/0
  • Абырвалг

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

    Spritz 23 сентября 2011 г. 7:39, спустя 1 день 15 часов 13 минут

    как думаете, что будет, если я на серваке размещу крякнутые продукты атлассиана?
  • disc

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

    Spritz 23 сентября 2011 г. 11:32, спустя 3 часа 52 минуты 41 секунду

    Абырвалг, будет profit! :)
  • adw0rd

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

    Spritz 23 сентября 2011 г. 12:43, спустя 1 час 11 минут 33 секунды

    Абырвалг, зачем тебе это? Скольким людям в твоей команде нужен доступ к продукту? Если <=10, то купи за 10$ в год и не парь мозги
    adw/0
  • Абырвалг

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

    Spritz 23 сентября 2011 г. 13:01, спустя 17 минут 40 секунд

    у Crucible это не 10 а 5 человек, а нас уже пятеро
    Спустя 11 сек.
    ищем шестого

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