Форум → Сообщество → Профессиональная деятельность → Взаимопомощь → Нужно решение для ревью кода
Нужно решение для ревью кода
Страницы: ← Следующая страница →
-
Сабж. Вот есть репо в меркуриале. Ребятки коммитят, мне что-то не нравится, я им значит пишу комментарии в строчках, которые мне не нравятся, они их видят, может быть даже имейлы отправляются сразу. Ну как на гитхабе.
Гитхаб не подходит тем, что там гит, и бесплатно он такого не предоставляет. Что-то типа redmine + code review плагин на нее, только это ж нужно устанавливать и настраивать редмайн, а у меня все времени нет. -
Сен. 20, 2011, 9:15 п.п., спустя 13 минут 4 секунды
а у меня все времени нет.
ребетишек запарь)Сапожник без сапог -
-
Сен. 20, 2011, 10:36 п.п., спустя 1 час 16 минут 54 секунды
ребетишек запарь)
да они не шарят такое. Набокопорят еще на сервакеhttp://hg-git.github.com/
та не, это дрюч туда-сюда гонять, +платить же ж еще нужно, у нас около 8 приватных репо -
Сен. 20, 2011, 10:47 п.п., спустя 10 минут 58 секунд
Абырвалг, http://www.atlassian.com/software/crucible/ в связке с jira оно работает, но может отдельно также куда то прикрутить получится. -
Сен. 20, 2011, 10:56 п.п., спустя 8 минут 50 секунд
да, продукты атлассиана крутые. Но боюсь, что это установить будет куда сложнее редмайна.
короче чувствую, что редмайн все же буду ставить, но вы все равно кидайте всякие тулзы/сервисы, интересно что есть в миреСпустя 47 сек.что интересно - вот битбакет от атлассиана тоже же. Но там комментарии чисто к коммиту в целом можно оставлять, но не к конкретной строке -
Сен. 20, 2011, 11:28 п.п., спустя 31 минуту 26 секунд
что интересно - вот битбакет от атлассиана тоже же. Но там комментарии чисто к коммиту в целом можно оставлять, но не к конкретной строке
а ты в комменте строку указывай и кусок кода -
Сен. 21, 2011, 10:22 п.п., спустя 22 часа 54 минуты 18 секунд
+1, сейчас его начинаем юзать, удобный
Абырвалг, http://www.atlassian.com/software/crucible/ в связке с jira оно работает, но может отдельно также куда то прикрутить получится.
+ pre-commit хуки для предварительного Code Review, проверка стилю, проверка формата комментариев и т.д., короче все что можно автоматизизироватьhttps://smappi.org/ - платформа по созданию API на все случаи жизни -
Сен. 21, 2011, 11:28 п.п., спустя 1 час 6 минут 21 секунду
я наверно буду ставить крякнутую версию+ pre-commit хуки для предварительного Code Review, проверка стилю, проверка формата комментариев и т.д., короче все что можно автоматизизировать
расскажи пожалуйста как это работает в связке с гитом (вы ж его используете?)? У каждого разработчика хук стоит что ли? -
Сен. 22, 2011, 12: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.
а гиту ни копейки не дали, вухаха, так ему и нада! -
Сен. 22, 2011, 3: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"https://smappi.org/ - платформа по созданию API на все случаи жизни -
Сен. 23, 2011, 6:39 п.п., спустя 1 день 15 часов 13 минут
как думаете, что будет, если я на серваке размещу крякнутые продукты атлассиана? -
-
Сен. 23, 2011, 11:43 п.п., спустя 1 час 11 минут 33 секунды
Абырвалг, зачем тебе это? Скольким людям в твоей команде нужен доступ к продукту? Если <=10, то купи за 10$ в год и не парь мозгиhttps://smappi.org/ - платформа по созданию API на все случаи жизни -
Сен. 24, 2011, 12:01 д.п., спустя 17 минут 40 секунд
у Crucible это не 10 а 5 человек, а нас уже пятероСпустя 11 сек.ищем шестого
Страницы: ← Следующая страница →
Пожалуйста, авторизуйтесь, чтобы написать комментарий!