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

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

Чем дольше код стоит на инспекции, тем больше шансов, что код в репозитории изменится, придется мерджить и вновь выставлять на инспекцию. Такой процесс может затягиваться на недели. Пока у нас нет стандартного решения этой проблемы и каждый подобный случай решаем индивидуально. Возникают ли у вас такие ситуации?
Мы для избежания таких проблем стараемся во-первых, ставить задачам по инспекции более высокий приоритет по сравнению с обычной разработкой, а во-вторых разрабатывать небольшими законченными кусками, что бы инспектору не приходилось неделями сидеть и разбираться что же там такое написали.
Мы решаем эту проблему так же, как описали здесь:

1. Мы реализуем новый функционал в отдельных ветках.
2. Каждая большая задача всегда разбивается на подзадачи.
3. Разработчик отдаёт код на инспекцию вменяемыми законченными частями.
4. Приоритет на инспекцию у нас ниже только критических уязвимостей.
Все верно. Не представляю как вообще можно мержить чью-то ветку, не просмотрем код и не прогнав тесты.

У нас в команде принято так:

До ревью отвечает за код разработчик, после ревью за возможные проблемы отвечает тот, кто просматривал и мержил код. В таком случае ответственность за свои действия на достаточно высоком уровне и народ перестает писать что попало.
Я правильно понял, что вы написали, что разработчик перестает нести ответственность за свой код после ревью? Так что тогда ему мешает писать код таким образом, лишь бы он прошел ревью, а если будет глючить потом, то и так сойдет?
Да, вы поняли верно. Но поскольку ревью делаю его коллеги, тоже такие же разработчики, с которыми он работает непосредственно каждый день, этот способ работает.
Ну люди бывают разные, и у некоторых экземпляров отношение к работе наплевательское.
Хотя таких, конечно, лучше сразу увольнять…
У нас всё проще — в системе контроля версий стоит pre-commit hook, который проверяет наличие строки «reviewer: %username%» или «pending-reviewer: %username%». При её отсутствии коммит не проходит.

Соответственно, если изменения кода существенны, ты зовёшь на ревью человека, который хорошо разбирается в коде, с которым ты работал. Если же это какие-нибудь незначительные поправки — вписываешь этого человека как pending-reviewer, а затем с помощью FishEye генерируется post-commit review.
НЛО прилетело и опубликовало эту надпись здесь
Ага. Контроль версий — SVN.
НЛО прилетело и опубликовало эту надпись здесь
Есть сборочный сервер (Bamboo), несколько раз в сутки собирает всё и гоняет тесты. А поверхностно убедиться можно и запустив программу/тесты локально.
CodeCollaborator — отличная штука. На прошлом проекте купили лицензию и с удовольствием пользовались.

Справедливости ради стоит отметить, что если ваша система контроля версий поддерживает shelving (как, например, Perforce), то ревью кода можно проводить и просто через электронную почту, эффективность это не слишком убавляет.
Мы в команде практикуем pre-commit review: написал код -> сделал патч -> отправил на группу (email) -> получил два одобрения -> commit. По патчу, однако, сложно судить об общей логике изменений. Лучше всего проверяется стиль кодирования :) Но и ошибки находятся, так что практика однозначно полезная.
pre-commit review на практике оказывается уж очень напрягающим
Мы просто задачу из in progress перевешиваем в review, и только потом в test/done. review проходят обычно на следующий день, в редких случаях — через неделю.
А насколько это удобно работать через патчи если quilt не используется? Ну то есть — есть таск в трекере, кто то сделал бранч, поработал, получил патч, выложил в трекере, его посмотрел ревьювер, ок, потом запушили. Не очень удобно для не больших проектов.
Вот другой случай — идет большая монотонная работа по разработке, дескритизация тасков низкая. Хочется ревьювить ежедневные комиты. Чем удобно пользоваться?
Мне кажется что мы с Колей Алименковым достаточно хорошо раскрыли тему практики Code Review. Не буду много писать — если интересно, посмотрите запись нашего доклада на AgileDays 2011 vimeo.com/22736643
Исходя из заголовочной картинки, Code Review оставит вас без штанов!
Зарегистрируйтесь на Хабре , чтобы оставить комментарий