Исправление Reset, (текущая версия) :
На code review обсуждается код всех коммитов фичи целиком, очевидно же.
Не очевидно. code review это ТОЛЬКО ревью коммита.
Потому этот код, внезапно, логически связан.
И что? У тебя фича может занимать в конечном варианте 1000 строк, 10000 строк и т.п. Если делать ревью как ты говоришь, то в транк она попадет через полгода, год и т.п. Я это всё уже проходил. Этот подход не работает.
И чо?
И то. Вся архитектура и взаимосвязи кода обсуждены на этом этапе и подготовлен дизайн-док. Когда будешь смотреть отдельные коммиты, то можешь свериться с доком.
Исходная версия Reset, :
На code review обсуждается код всех коммитов фичи целиком, очевидно же.
Не очевидно. code review это ТОЛЬКО ревью коммита.
Потому этот код, внезапно, логически связан.
И что? У тебя фича может занимать в конечном варианте 1000 строк, 10000 строк и т.п. Если делать ревью как ты говоришь, то в транк она попадет через полгода, год и т.п. Я это всё уже проходил. Этот подход не работает.
И чо?
И то. Вся архитектура и взаимосвязи кода обсуждены на этом этапе и подготовлен дезайн-док. Когда будешь смотреть отдельные коммиты, то можешь свериться с доком.