Здравствуйте, CreatorCray, Вы писали:
R>>Какое нахрен наслоение многочисленных мёржей?
CC>А такое. Тут чуть чуть поменяли, тут чуть чут добавили. Потом смотришь на это всё и понимаешь где можно что выровнять, что то вынести и сделать более generic.
CC>Инкрементальные изменения они такие.
Изменения — инкрементальные, а смотреть нужно всегда на общую картину.
R>> Мёрж, даже если в нём нет конфликтов, требует ревью.
CC>Ревью всегда фокусируется на изменениях а не на более общей картине.
Нет, это абсолютно неверно. Нужно быть достаточно запущенным наркоманом, чтобы интересоваться правками в отрыве от того, к чему эти правки применяются. Правки сами по себе не работают. Любые правки должны переводить систему из одного адекватного состояния в другое — и на код ревью смотреть нужно в первую очередь на это.
R>>Как определяете, что требования устаканились?
CC>Строго на глаз, есессна.
Ясно