Форум на Исходниках.RU https://forum.sources.ru Форум на Исходниках.RU Форум на Исходниках.RU https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2959626 Thu, 11 Aug 2011 13:01:36 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2959626 Вуйко з полонини Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2915568 Fri, 17 Jun 2011 05:33:25 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2915568 Надо курить в эту сторону.]]> Мохнатый Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2915550 Fri, 17 Jun 2011 04:34:12 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2915550 Как я понял, это что то вроде стартапа, правильно?]]> Бобёр Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2912650 Fri, 10 Jun 2011 21:11:52 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2912650
у нас на работе на каждом проекте сидит человек, если проект большой, то стараются делить проект на части по людям, каждый отвечает за свое.

Я хочу внести изменения, что бы работа проводилась командами по 2-3 человека, один из них будет ведущим, который и будет делать коде-ревью.

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

Code Collaborator посмотрел, но что-то он тяжеловат для подобной простой задачи. У нас в подразделении не более 8-ми человек и планируется организовать пробную команду из 3-х человек, вот хочется в этой команде и организовать коде-ревью.

Кстати, может посоветуете что почитать хорошего по поводу организации команд и т.п.?]]> sploid Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911777 Thu, 09 Jun 2011 19:53:56 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911777 Цитата
Вот... Меня терзают смутные сомнения. На критическом пути ответственных должно быть минимум. Иначе непонятно, с кого спрашивать, если сборка развалилась. В общем, тут всё не так просто. :)


Я там в клубе ссылочку давал, русский перевод концепии "Programming, Mothefacker" ;)

Цитата
Если продакшен-кода - десятки/сотни мегабайт, коммитеры исчисляются десятками или сотнями, то один "ответственный" физически не сможет ни за что отвечать. ;)


Ну, в исходной задаче не стояла цель "закоммитать сразу везде и т.д.", человек явно работает над каким то одним проектом.
А у вас всё собирается "в одной большой куче" что ли? Т.е. ночью билд система берёт, выгружает самые последние исходники "на сегодня" в одну кучу и пытается их собрать? Обычно же явно указываются, скажем, точная версия такой то подсистем X, Y, P, которая нужна для сборки проекта F. Как оно может не собраться в такой ситуации?

Вот, кстати, git могу ещё раз прорекламировать - у тебя есть проект, есть к нему субмодули (субмодули хранятся в своих репо, и разрабатываются отдельно), ты указываешь точную версию подсистемы которая нужна В ТВОЁМ ЛИЧНОМ ПРОЕКТЕ. Если самая свежая версия подсистемы тебя не устраивает, ты просто её не берёшь. А если обновляешь - тупо проверяешь её работоспособность.]]> Бобёр Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911476 Thu, 09 Jun 2011 13:53:07 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911476 nastenka Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911455 Thu, 09 Jun 2011 13:32:36 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911455 Цитата Машина @
Ну желательно острая иерархия, чтобы равнозначные особы не спрашивали друг у друга разрешения закоммититься в рабочую ветку. Только старший с младшим :rolleyes:

Вот... Меня терзают смутные сомнения. На критическом пути ответственных должно быть минимум. Иначе непонятно, с кого спрашивать, если сборка развалилась. В общем, тут всё не так просто. :)]]> Flex Ferrum Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911452 Thu, 09 Jun 2011 13:28:00 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911452 Цитата Flex Ferrum @
Если продакшен-кода - десятки/сотни мегабайт, коммитеры исчисляются десятками или сотнями, то один "ответственный" физически не сможет ни за что отвечать.

Ну желательно острая иерархия, чтобы равнозначные особы не спрашивали друг у друга разрешения закоммититься в рабочую ветку. Только старший с младшим :rolleyes:]]> Машина Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911448 Thu, 09 Jun 2011 13:24:22 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911448 Цитата Бобёр @
Ну, как бы если есть ОДИН ответственный за продакшн (он же - старший программист из постановки задачки), то холиваров возникнуть не должно.

Если продакшен-кода - десятки/сотни мегабайт, коммитеры исчисляются десятками или сотнями, то один "ответственный" физически не сможет ни за что отвечать. ;)]]> Flex Ferrum Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911441 Thu, 09 Jun 2011 13:18:10 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2911441 а вообще в принципе, например, для eclipse есть jupiter.]]> nastenka Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2910925 Wed, 08 Jun 2011 20:32:42 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2910925
Ну т.е. я вообще согласен что да, в оффлайне бывает по всякому, у нас вот местами доходит до того, что бранчи вообще не сливаются, и уже никогда не сольются, потому что это такой гемор.... Но в исходной постановке задачи таким способом её решить возможно.]]> Бобёр Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2910922 Wed, 08 Jun 2011 20:27:57 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2910922 Бобёр, тут имеется совершенно другой workflow. Нередко бывает, что ревью какого-нибудь интерфейса какой-нибудь подсистемы - это баталии, похлеще наших холиваров. Тут в "оффлайне" (анализируя разъехавшиеся бранчи) сделать это эффективно - достаточно сложно.]]> Flex Ferrum Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2910907 Wed, 08 Jun 2011 19:55:19 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2910907 Цитата
Например есть ведущий программист и просто программист.
Надо организовать что бы перед тем как просто программист закоммитит код в основную ветку в какой-то системы хранения версий исходников (какую использовать тоже вопрос, сейчас используется SVN), ведущий программист просмотрел эти изменения и принял их или сказал какие куски переписать.


Можно обойтись штатными средствами системы хранения версий исходников git.
В нём очень просто бранчеваться а потом мерджиться.
Выглядит процесс примерно так.
Перед реализацией какой то фичи человек отстреливает себе бранч, не нарушая работу остальных.
В своём бранче от может делать что хочет, в том числе и коммитить. Коммитить вообще полезно, если у вас, например, над рабочим компьютером периодически течёт с крыши (у меня вот текло зимой :) ). Потом код можно почитать, провести код ревью и сделать merge в основную ветку (даже если она успела уйти вперёд).]]> Бобёр Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2902384 Fri, 27 May 2011 11:10:25 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2902384 Code Collaborator. Он платный (наверняка есть бесплатные аналоги). Предназначен как раз для осуществления описанного тобою процесса. Может работать в связке с СКВ в том числе и в варианте, когда коммит не проходит, пока не закрыто ревью.]]> Flex Ferrum Version Control https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2902374 Fri, 27 May 2011 11:02:08 +0000 коде ревью перед коммитом https://forum.sources.ru/index.php?showtopic=332343&view=findpost&p=2902374
Например есть ведущий программист и просто программист.
Надо организовать что бы перед тем как просто программист закоммитит код в основную ветку в какой-то системы хранения версий исходников (какую использовать тоже вопрос, сейчас используется SVN), ведущий программист просмотрел эти изменения и принял их или сказал какие куски переписать.

Как это реализовать и какую систему хранения версий использовать?]]> sploid Version Control