Боты-напоминатели не работают

Посмеялись над отменой ревью пул реквестов, теперь о серьезном. Я работал в разных командах. Для одних ревью никогда не было проблемой. Для других — всегда было проблемой. Проблемой в том смысле, что пул реквесты ревьювились недостаточно быстро.

В некоторых командах это можно было отследить собрав данные. В других были люди, которым требовалось, чтобы их пулы проверяли БЫСТРЕЕ.

В обоих случаях проблему пробовали решить автоматизацией.

Считали, что если с утра людям подсунуть список, то они тут же начнут проверять код. Нет. Вернее, первые пару дней люди действительно реагировали на бота. Потом, как с рекламой, люди переставали обращать внимание на сообщения.

Это никогда не проблема автоматизации. Это никогда не проблема того, что люди “забывают”. Это проблема культуры команды или разницы темпераментов.

Разницу темпераментов вылечить “нельзя”. Это как с порогом терпимости к срачу вокруг. Если ваша терпимость ниже, чем у ваших супругов — вы будете страдать и ругаться. Человек ожидает, что код, который он отправил до обеда, посмотрят к вечеру. Остальные люди смотрят код только с утра. Человек начинает думать, что его игнорируют и команда вообще его не ценит. Либо, что он один все делает, а другие балду пинают. На самом деле это только его проблема, у всех остальных код ревьювится, тикеты закрываются. Все деливерят и счастливы. Человеку можно попробовать поработать с тревожностью, если задержка в ревью бьет по его самолюбию. Либо найти другую команду, если кажется, что все тупо тормозят. Иначе человек заебет всех вокруг, но все равно будет несчастен.

Если никто особо не жалуется, но код висит неделями и тикеты не закрываются, то это уже проблема культуры. И я не знаю, как ее решать в общем случае. Нужно смотреть, что блокирует ревью и убирать блоки. Возможно код очень тяжело тестировать, а вы требуете, чтобы проверяющий тестировал код. Возможно у вас 100 репозиториев, но пул реквесты не привязываются к задачам в жире. Возможно ваши тесты постоянно сломаны и люди пытаются уговорить CI поставить зеленую галочку прежде чем смотреть код. Возможно вы наваливаете задачи в спринт и привязываете их к конкретным людям. В итоге люди стараются разгребать быстрее свое, чтобы не огрести. Возможно все вообще не имеет никакого смысла и люди уже давно просто выключились из процесса. Все счастливые команды счастливы одинаково. Все несчастные несчастны по-своему.

Как вы решаете проблему медленных ревью? Отменяете их совсем? Заводите доску позора плохих ревьюверов? Даете премии хорошим?