Проверка кода — одна из наиболее высокодоходных деятельностей TL: она защищает качество и учит. Цель — лучший код и более сильная команда, а не контроль доступа. Проверки должны быть быстрыми, добрыми и сосредоточенными на том, что имеет значение.
Принципы
text
✓ Review the CODE, never the person ("this function" not "you")
✓ Distinguish must-fix from nice-to-have (label nits explicitly)
✓ Ask questions, don't issue commands ("what about X?" invites discussion)
✓ Praise good work, not just problems
✓ Be fast — a PR blocked for two days kills momentum
✓ Approve when it's good enough, not perfect
Конкретный пример
Вместо «Это неправильно, используй map» напиши: «пометка: map здесь избежит вложенного цикла, стоит ли? Не блокирую.» Та же суть, но учит, оставляет место для несогласия и не блокирует слияние.