тогда что такое в вашем поинмании "Соответствие ожиданиям" ?

Какое из слов непонятно?

непонятно какой смысл вы вкладываете в понятие "ожидание" и фразу "соответствие ожиданиям", если это не связано с фичами и задачами

Дак поставьте его решать задачу вместе с опытным. Пусть спроектируют вместе всю фичу. Разбейте задачу на подзадачи и пусть о делают их параллельно. В ходе разработки будут появляться вопросы - пусть спрашивает. Работают в паре -  значит придётся мержить код в мастер и так чтобы не ломалось - научитесь делать вечноживой мастер

Лучше, конечно, Саймона почитать, но хоть так. Два последних абзаца.

Очевидно, чтобы вести себя иным образом, надо делать, много раз правильно и не делать много раз неправильно. Если не дать правильные упражнения, то никакого значимого результата не будет. Можно, конечно, переоткрывать все самому, но это же не эффективно, а ревнители кодревью ратуют за "эффективность"

Так все что вы говорите правда, но еще бывает полезно дать возможность спкойно посидеть над кодом и подумать

Что там непонятного. Соответствие ожиданиям «яжтехлида». Никакого отношения к качеству ревью не имеет. Есть власть, власть знает как надо, власть требует чтобы было сделано как она хочет. Не нравится, пшел вон.

так конечно можно бросаться, но лучше всетаки получить трактовку с той стороны, чтоб было чего обсуждать, а то както непредметно и без пользы

Хочется повысить качество, ставим линтеры, статические анализаторы, лимиты на инкремент тестового покрытия, внедряем практику проработки дизайна до написания кода. Сложно, долго, надо думать.

Хочется самоутвердится, декларируем что работаем по GitFlow, ставим вахтера на мерж в мастер, пишем бумажку с чек листом. Дешево, быстро, не надо думать.

О, опять про баню разговоры 😂

"у нас работает!"

Накину свежего - скл-код ревьювить надо почти всегда. Так как очень мало разрабов умеют в него. 😏

Предметно я уже высказывался. Код ревью нужен чтобы пропускать только те изменения которые нравятся мэйнтейнеру и отклонять те что не нравятся, вне зависимости от качества и ценности. На этом весь ОпенСоурс держится.

И второй вариант, код ревью не помешает в случае если код может убить человека, но проверка не поддается автоматизации.

оооооооо, охохохох, тут точно надо обучение, а не ревьюить

Это легко автоматизируется. Проверка SQL

Как по мне, важная вещь, которую обеспечивает код-ревью - это читаемость и поддерживаемость кода. Я-как-кодер по своему опыту знаю, что даже в собственном коде через полгода разобраться бывает сложно, а всё же какой-никакой контекст к нейронам прилип. Разбираться в чужом - сложная задача с высокой когнитивной нагрузкой, поэтому лучше эту нагрузку постараться снизить заранее. А лучше способа, чем дать коллеге прочитать и разобраться - наверное и не бывает.

Ну какое обучение, это все сплошь сениоры с мнением, что кодревью скл не нужен. )

что именно автоматизируется?