Zada Sep 14 2010 at 23:12

Ревью кода в mercurial

3 min

4.6K

Open source*

+45

Comments 21

UFO landed and left these words here

Zada Sep 15 2010 at 02:19

Вот не стоит смешивать 2 совершенно разные веши.
Мне теперь не понятно, люди считают, что код ревью прерогатива issue tracker'а или же им не нравится слово ложить

По первому — мне кажеться совершенно обратное. Код ревью, это для повышения качества кода, для обмена опытом, для ревью кода в конце-концов. Т.е это как бы сугубо часть для разработчиков. Нечего сюда мешать кастомеров и менеджеров, которые могут на основе всего этого делать не правильные выводы.
Ну и у такой организации есть приемущество, все ходит вместе с исходниками и все синхронизируется в понятно виде.
Еще, найти вменяемый инструмент для код ревью, который бы был хоро интегрирован в issue tracker — задача не из простых. На удивление мало их, да убоги они.

le0pard Sep 15 2010 at 00:27

Локальный bitbucket.org?

Zada Sep 15 2010 at 00:27

Нет, это совершенно другое.

rojer Sep 15 2010 at 01:10

codereview.appspot.com
вы можете завести себе свой собсвенный инстанс — code.google.com/p/rietveld/
работает также и без AppEngine — нужен Django + один модуль.

Zada Sep 15 2010 at 02:13

Круто, а какие основные приемущества перед hg review?

rojer Sep 15 2010 at 13:08

я hg review не видел, но судя по скриншотам rietveld посимпатичнее будет.

youmee Sep 15 2010 at 02:10

а что это у вас за изображение такое в начале топика, прозрачное?) статистику ведете?)

Zada Sep 15 2010 at 02:14

Да.
Не только в начале, но и в конце!

googol Sep 15 2010 at 05:53

У гита есть Gerrit.

Вот пример работающий https://review.source.android.com

dmiceman Sep 15 2010 at 07:03

Эх, жаль только что он с hgwebdir.cgi не завязан.

Zada Sep 15 2010 at 11:27

Почему жаль?

dmiceman Sep 15 2010 at 22:42

Ну как почему? Неудобно это — смотреть туда, смотреть сюда, да и багтрекер еще где-то сбоку прилеплен. Вот бы тут один интерфейс…

Zada Sep 15 2010 at 23:34

Эту штуку не надо запускать в паблик.
Эту штуку надо запускать локально, каждому разработчику, персонально.

tenitski Sep 15 2010 at 13:16

Очень рекомендую ReviewBoard (http://www.reviewboard.org/).
Достаточно мощный инструмент для просмотра и ругания кода.

Из меркуриала код передается в ReviewBoard через
hg postreview

Murkt Sep 15 2010 at 15:43

Им очень неудобно пользоваться, ИМХО. Он навязывает свои стратегии использования, которые очень часто не согласуются с тем, что по факту происходит.

tenshi Sep 17 2010 at 00:59

лучшее ревью — добавить коммент в код. ведь это же не бесполезный флейм? тогда когда кто-то будет редактировать этот код — он сможет действовать в соответствии с указанными замечаниями.

Azy Sep 20 2010 at 21:46

но тем не менее часто нужно зафиксировать кто код смотрел
не писать же каждый раз это в коде

tenshi Sep 21 2010 at 02:21

это зачем? если нужно что-то донести до всех — есть рассылка. если только тем, кому с ним будет работать — он и так посмотрит.

Azy Sep 21 2010 at 11:57

Это в зависимости от процессов компании. Иногда есть строгие инструкции по обязательному рецензированию кода.

tenshi Sep 21 2010 at 12:19

мы эти процессы как раз и обсуждаем. нэ?