>>203160
>Ну, во-первых, ревью нужено хотя бы для того, чтобы поглядеть, чего там коллеги накуролесили. Чтобы потом не оказалось неожиданностью.
>Во-вторых, его отдают как правило сеньёру или ещё какому главнюку.
Получается, код-ревью нужен сеньору (тимлиду), чтобы оставаться сеньором (тимлидом), так? А остальные, не проводящие в код-ревью, играют в царя горы с перечитыванием лога или каждый раз матерятся на чужие изменения.
>В нормальных проектах есть соглашения о форматировании, неймингах и тому подобном.
Это-то понятно. Я говорю как раз о случаях вне соглашений, когда человек суёт в код-ревью именно своё субъективное мнение. Просто каких-то регламентов на код-ревью мне видеть не приходилось. Есть ли такая практика - составлять регламенты код-ревью?
И, что самое интересное, мнение на счёт того, каким должен быть хороший (читаемый etc) код, у всех разное. Почему-то каждый это своё мнение в процессе ревью пропихивает как непреложную истину.