Комментарии 3
Самое эффективное ревью - это то которое делаешь самостоятельно, на утро следующего дня. За ночь глаз отвыкает от кода и видишь свои ошибки сам. Так что иногда код должен просто "настояться")
Раньше до ИИ было, что изменения в код добавлялись медленнее, и в найтли сборке было легче заметить изменения и найти причину.
Вдобавок людям лень писать повторяющийся код, и они начинают его как-то реорганизовывать и более лаконично писать, а ИИ это чуждо и он может однотипных изменений хоть на тысячу строк сделать. И, к сожалению, как будто теперь вместо простого и организованного кода, его теперь просто больше и он хуже качеством.
Получается что-то вроде трагедии общин - каждому разработчику проще нагенерить код ценой потери качества, но в итоге получается раздутый проект с техдолгом, в котором всё перманентно разломано и все страдают.
P.S. А про слои ревью - полностью согласен. В предыдущей команде можно было влить изменения в мастер и потом создать ревью - было очень удобно. Сейчас требуется обязательный аппрув ревью человеком + долгое и очень нестабильное CI, в сумме это ад. Сначала ждёшь ревью от человека, потом он просит что-нибудь поправить (один слой), потом CI перезапускает тесты и зачастую падает. И эти тесты - ещё один слой, причём там бывают и тесты нестабильные, и просто поломанные кем-то другим, когда только ребейз поможет. Всякие мелочи даже исправлять не хочется, код пишется за минуты и потом тратятся дни, чтобы его влить.
Еще можно посмотреть на свои творения недели через две, а то и через месяц.
Сразу становится понятно, как нужно было назвать переменные, по каким сервисам разбить код, какой алгоритм оптимальнее для решения поставленной задачи, да и по поводу постановки возникают интересные мысли (а постановщик тоже куда-то торопился)

Каждый слой ревью замедляет работу в 10 раз