?

Log in

November 2016   01 02 03 04 05 06 07 08 09 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30
cartoon

Миф о документации, продолжение

Posted on 2011.06.18 at 23:06

Comments:


Gaperton
gaperton at 2011-06-18 20:21 (UTC) (Link)
> Фатально помешать ревью может отсутствие двух вменяемых программистов в команде.

Нет, не может. Ревью делается по правилам, а не по мнениям. Ошибка ревью - либо человек багу нашел, либо несоответствие кодстандарту.

Если есть претензии к читабельности - отправляем на ревью ко второму программисту, и смотрим, бьются ли их рекомендации между собой.

Конфликт ревью разрешает тимлид под свою ответственность.
Яков Сироткин
yakov_sirotkin at 2011-06-18 20:31 (UTC) (Link)
На некоторых проектах бывает только один программист. И никакого тимлида, который читает код. И даже если программиста всё-таки два, то это очень не просто может быть наладить код ревью из-за разных человеческих особенностей.
Gaperton
gaperton at 2011-06-18 20:38 (UTC) (Link)
> На некоторых проектах бывает только один программист.

Что у нас бывает с некоторыми проектами, когда этот один программист крепко болеет, увольняется, или ему не дай бог падает на голову кирпич?

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

> И никакого тимлида, который читает код.

Проверка должна быть перекрестной, а не тимлидской. И код тим-лида тоже должен кто-то проверять из его команды - без каких-либо исключений.

> И даже если программиста всё-таки два, то это очень не просто может быть наладить код ревью из-за разных человеческих особенностей.

Это, право слово, очень просто. Достаточно фамилию проверяющего в текст коммитмессаджа писать, и реально наказывать за вопиющие косяки обоих. Сразу всем станет очень просто.
Яков Сироткин
yakov_sirotkin at 2011-06-18 20:53 (UTC) (Link)
Если я ухожу с проекта, то за мной код правят другие программисты, мне даже вопросов не задают. Я правда стараюсь понятно писать.

Никакого тимлида - значит тимлида нет физически. При этом два программиста могут работать в совсем разных компаниях.
Previous Entry  Next Entry