?

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:


hedgeov
hedgeov at 2011-06-19 19:16 (UTC) (Link)
Если есть соглашение "тикет на отдельной ветке" -- проверяющему высылается имя ветки, которую надо смотреть. При этом процесс может выглядеть так: Тикет имеет состояния "решено", "ревьюировано" и "интегрировано". Ревью имеет свой собственный жизненный цикл с состояниями "создано", "состоялось", "отклонено", "исправлено", "закрыто". Ревью разрешается инициировать когда у тикета состояние "решено". Ревью создается, при этом указывается имя ветки, в которую надо смотреть. После того как ревьюер посмотрел в указанную ветку и налабал массу замечаний, ревью считается "состоявшимся". После того как автор все замечания устранил на той же самой ветке, ревьюер смотрит на результат и либо меняет состояние на "исправлено", либо на "отклонено". Если "отклонено", то автор продолжает исправлять, пока ревьюер не согласится. Если состояние "исправлено", то автор закрывает ревью (переводит в состояние "закрыто"). После этого, когда тикет в состоянии "исправлено" и ревью в состоянии "закрыто", можно мержить изменения с ветки в trunk, после чего переводить тикет в состояние "смержено". Для перевода в состояние "смержено" обязательно указывается номер ревью.
melkus
melkus at 2011-06-20 08:33 (UTC) (Link)
В случае если фикс занимает 10 минут, что иногда встречается, проще позвать человека к своему монитору. Ваш вариант мне нравится когда я начинаю думать о больших изменениях, а использовать ветки для багов и мелких тикетов мне кажется расточительством.
hedgeov
hedgeov at 2011-06-20 09:13 (UTC) (Link)
Я за унифицированный процесс вне зависимости от сложности изменений. "В каждом безобразии должно быть единообразие" -- когда процесс один и без исключений -- его проще запомнить и применять. Опять же, не всегда ревьюер физически способен к монитору подойти (из другой страны он например). На самом деле не так много времени это веткотворение занимает.

Самый большой плюс веток на мой взгляд -- простота разбора коллизий, возможность быстро и безболезненно выкинуть кривой багфикс если что-то пошло не так. Если все изменения на mainline -- просто так середину не выкинешь и придется сравнительно долго согласовывать между авторами откаты и перевнесение изменений.
Previous Entry  Next Entry