Comments 3
цикл "Починили -> Поменяли -> Сломалось -> Починили..." очень знаком )))
Я думаю, ревью это не тот этап, на котором нужно вспоминать, что оказывается, нужно обучение и обмен опытом. Это должно делаться где-то гораздо раньше - на внутренних митапах, в сессиях параллельного программирования и т.п. А не блокировать все тогда, когда весь код уже написан, и задача ждёт только апрува.
Согласен, в каких то случаях ревью не поможет. Но в моем контексте, когда для джунов таски нарезаны старшим, то по сути проверяется сделал ли так, как договаривались.
Когда новичок приходит в компанию, ему намного легче влиться и понять какие есть договоренности через код ревью. Еще есть документация, нужно как то понять, что новичок делает по ней, а не как привык делать раньше.
Парное, тренинги - есть. Даже дизайн ревью решения иногда. Но как понять, что все они не были бесполезными? Ревью - это отличная входная точка для парной работы. У нас вышло, что небольшая блокировка в ревью, снизила количество доработок, что привило к снижению итогового времени.
Почему системы никогда не будут стабильными