@entech на самом деле надо было мне чуть больше контекста добавить. Код ревью изменений в коде может выполняться разными проверяющими: от ‘первый раз вообще вижу этот код’ до ‘вместе прорабатывали решение, очень хорошо понимаю о чем идёт речь’.
Моё сообщение было больше про второе. Как раз принимается активное участие и в исследовании, оценке и приоритизации. Абсолютно правильное утверждение насчет стилистики и т.п. - тут уже давно все автоматизировано.
С чем несогласен:
“а готов ли сам с этим кодом возиться, если надо будет разобраться с чем-то не работающим или бажным?” - если такой код попадает на ревью - я считаю что он просто не до конца подготовлен и не должен доходить до ревью. Ожидается, что в команде есть некоторые соглашения по качеству кода, стилю, подходам.
Я в последнее время больше концентрируюсь на том всё ли сделано оптимально, правильно ли выбрана структура данных, нет ли лишних аллокаций памяти, правильный ли выбран алгоритм.