День 2550. #Карьера
Топ Советов по Повышению Продуктивности. Часть 5
Части 1, 2, 3, 4

5. Правило 2 минут для проверок кода
Проверки кода — место, где умирают благие намерения. Как обычно бывает: кто-то открывает пул-реквест с 47 измененными файлами и 2300 строками. Вы видите уведомление и думаете: «Я проверю это позже, когда будет время». «Позже» никогда не наступает. PR лежит три дня. Автор пишет вам. Вы наконец выделяете час, открываете PR, сразу чувствуете себя перегруженным, бегло просматриваете его, одобряете с комментарием «Выглядит неплохо», и возвращаетесь к своим делами. Так вы сами способствуете той же проблеме, которая вас раздражает, когда вы ждёте проверки ваших PR.

Правило 2 минут
Если проверка кода займет меньше 2 минут, сделайте её немедленно.
Не «когда закончите текущую задачу». Не «после этой встречи». Не «в назначенное для проверок время». Немедленно.

Небольшие проверки выполняются быстро. PR на 30 строк с чётким контекстом можно проверить за 90 секунд. Прочитать код, убедиться, что он понятен, оставить комментарий или одобрение — готово.

Когда люди знают, что небольшие PR проверяются мгновенно, а большие остаются в очереди бесконечно, поведение меняется органично. Команда естественным образом начинает разбивать работу на более мелкие, более удобные для проверки части.
Это снижает и вашу когнитивную нагрузку. Вместо того чтобы 12 ожидающих проверок преследовали вас, как технический долг, вы сразу же выполняете небольшие. Чувство вины за то, что вы являетесь узким местом, исчезает.
Это улучшает качество кода. Быстрая обратная связь означает, что разработчики работают над проектом, пока контекст свеж. Они все ещё думают о проблеме. Они не переключились на три другие задачи. Раннее выявление проблем делает их исправление дешевле.

Но вот в чем загвоздка: небольшие PR должны быть легко идентифицируемыми.

Работайте с командой над установлением правил:
- PR менее 200 строк получают метку «small».
- Используйте префиксы заголовков PR: [SMALL] или [QUICK].
- Настройте уведомления в Slack или по email, которые будут отправляться по-разному для небольших и больших PR.
- Составьте командное соглашение: небольшие PR проходят проверку в тот же день, большие — когда будет время.

Для PR, занимающих более 2 минут, выделите время в календаре для работы по проверке кода. Относитесь к этому как к любой другой важной задаче. Но не позволяйте большим проверкам мешать вам немедленно выполнять быстрые.

Когда команда коллективно внедряет это, скорость проверки резко возрастает. Эта тесная обратная связь ускоряет всё. Функции выпускаются быстрее. Знания распространяются более равномерно. Ошибки обнаруживаются раньше. Сотрудничество ощущается не как бюрократия, а как настоящая командная работа.

Самое сложное — остановить текущую работу, чтобы что-то проверить. Мозг сопротивляется переключению контекста. Но двухминутная проверка — это кратковременное переключение внимания, которое обеспечивает движение вперёд.

Советы
- Сделайте уведомления о PR видимыми (Slack, email, расширение для браузера);
- Используйте мобильное приложение GitHub или GitLab для просмотра небольших PR во время перерывов на кофе;
- Создайте сочетание клавиш для быстрого перехода к очереди проверок;
- Отслеживайте время ответа на проверки PR в течение недели — осведомлённость приводит к улучшению;
- Создайте шаблоны PR, которые поощряют просмотр небольших PR. Включите пункт в контрольный список: «Этот PR содержит менее 200 строк или разбит на несколько». Сделайте осознание размера частью культуры команды.

Накопительный эффект быстрой обратной связи — одно из самых эффективных улучшений производительности, которых может добиться команда. Правило двух минут — это не просто проверка кода, это целая философия. Уменьшите препятствия для быстрых действий, и вы будете совершать их чаще. Сделайте отзывчивость простой, и вы сами станете отзывчивыми.

Источник: https://dev.to/thebitforge/top-10-productivity-hacks-every-developer-should-know-151h