СА
Я уточнил у вас, работали ли вы на проектах с кодбазой в 50к+ LOC. Это валидный вопрос, потому что сейчас в моде микросервисная парадигма, где кодбазы находятся на уровне 5k-10k LOC.
Size: a a a
2021 May 08
VF
Ну так бы и написали
AD
А ещё попробуйте взять пивко-кофейко и посмотреть последние 20-30 пуллреквестов вашей команды. Там действительно стоящие неповторяющиеся замечания, которые решили реальную проблему и стоят потраченного на них времени?
СА
Я задал вам встречный вопрос: вы аргументируете против код ревью, но при этом спрашиваете меня, зачем делать четвёртое код ревью после трёх других. Этот вопрос противоречит отстаиваемой вами позиции.
VF
При этом, как я указал выше, полный размер проекта не особо интересен, потому что при нормальном дизайне, вне зависимости от сервисов и микросервисов, код любого модуля будет разумного размера
VF
Если вы заметили, я предложил рассмотреть гипотетическую ситуацию.
СА
Вы почему-то считаете, что "в коде можно разобраться за конечное время" - это свойство автора кода. А я утверждаю, что это практика, которую требуется структурно создавать и поддерживать.
AD
Ага, супер! Так может это про карту компетенций и бас факторы, а не кодревью?
VF
Ну есть же синтез, анализ и вот это вот все, давайте разбираться.
СА
В гипотетической ситуации, когда изменение кода, которое решает поставленную в тикете задачу, описанную в спецификации, прошло три ревью - четвёртое, разумеется, не нужно. Но я очень плохо представляю себе реальную ситуацию, когда такое происходит с реальными задачами на реальном проекте.
AD
Божечки, он не про конкретное кодревью
VF
Господи, не одна задача прошла три ревью. Три любых задачи одного человека прошли ревью от вас персонально.
AD
Типо если вы три раза посмотрели, что в холодильнике нет пожрать, зачем смотреть четвёртый раз
СА
Потому что холодильник наполняет целая команда, а не я один)
AD
Ладно, тупой пример, простите)
VF
Человек всегда ставит молоко на нужную полку, вы три раза проверили, зачем поверять четвертый и все последующие разы?
AD
Если ты прочитал отзывы о компании и понял что они норм и не мошенники, ты не будешь мониторить это каждую неделю
AK
Все равно ему нельзя доверять. Иначе как объяснить продолжение практики код-ревью в таком случае? говорю же Zero-trust
СА
Я уточню: если три разных члена команды трижды посмотрели, как конкретный человек решает поставленные задачи, высказали свои замечания, они были учтены - вы считаете, что в четвёртый раз нет необходимости смотреть?
AD
А если не было замечаний?