Здравствуйте, rosencrantz, Вы писали:

R>Изменения — инкрементальные, а смотреть нужно всегда на общую картину.
На ревью смотрят на диффы.
Ну и когда делают изменения то тоже стараются свести изменения к необходимому минимуму.
Рефакторинг банально геморно мержить когда в полёте куча других изменений, которым твой рефакторинг сломает merge.

R>Нет, это абсолютно неверно.
R>в отрыве от того, к чему эти правки применяются.
Ты опять не понял про что я.

R>>>Как определяете, что требования устаканились?
CC>>Строго на глаз, есессна.
R>Ясно

У нас банально есть стадия когда принимаются только багфиксы, и то не всё подряд, а только на одобренные баги, чтоб не было "переименуем порося в карася" т.е. фичу назовём багфиксом.
Хочешь что то в фиче поменять — свободен, вали в следующий релиз. Твой pull request просто интеграторы не примут и в мастер не смержат.

Есессна к этому моменту всё должно быть готово, так что посылать в пешее эротическое всяких ходоков с хотелками начинаем задолго до, исходя из реалистичности сроков.
Так что да, именно строго на глаз.