Как стать автором
Обновить

Комментарии 21

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

По первому — мне кажеться совершенно обратное. Код ревью, это для повышения качества кода, для обмена опытом, для ревью кода в конце-концов. Т.е это как бы сугубо часть для разработчиков. Нечего сюда мешать кастомеров и менеджеров, которые могут на основе всего этого делать не правильные выводы.
Ну и у такой организации есть приемущество, все ходит вместе с исходниками и все синхронизируется в понятно виде.
Еще, найти вменяемый инструмент для код ревью, который бы был хоро интегрирован в issue tracker — задача не из простых. На удивление мало их, да убоги они.
Локальный bitbucket.org?
Нет, это совершенно другое.
НЛО прилетело и опубликовало эту надпись здесь
Круто, а какие основные приемущества перед hg review?
НЛО прилетело и опубликовало эту надпись здесь
а что это у вас за изображение такое в начале топика, прозрачное?) статистику ведете?)
Да.
Не только в начале, но и в конце!
У гита есть Gerrit.

Вот пример работающий https://review.source.android.com
Эх, жаль только что он с hgwebdir.cgi не завязан.
Почему жаль?
Ну как почему? Неудобно это — смотреть туда, смотреть сюда, да и багтрекер еще где-то сбоку прилеплен. Вот бы тут один интерфейс…
Эту штуку не надо запускать в паблик.
Эту штуку надо запускать локально, каждому разработчику, персонально.
Очень рекомендую ReviewBoard (http://www.reviewboard.org/).
Достаточно мощный инструмент для просмотра и ругания кода.

Из меркуриала код передается в ReviewBoard через
hg postreview
Им очень неудобно пользоваться, ИМХО. Он навязывает свои стратегии использования, которые очень часто не согласуются с тем, что по факту происходит.
лучшее ревью — добавить коммент в код. ведь это же не бесполезный флейм? тогда когда кто-то будет редактировать этот код — он сможет действовать в соответствии с указанными замечаниями.
но тем не менее часто нужно зафиксировать кто код смотрел
не писать же каждый раз это в коде
это зачем? если нужно что-то донести до всех — есть рассылка. если только тем, кому с ним будет работать — он и так посмотрит.
Это в зависимости от процессов компании. Иногда есть строгие инструкции по обязательному рецензированию кода.
мы эти процессы как раз и обсуждаем. нэ?
Зарегистрируйтесь на Хабре , чтобы оставить комментарий

Публикации