- Всегда помни, что цель ревью - качественное выполнение конкретной задачи. Работаем с новым кодом, не ищем ошибок в легаси (для этого есть задачи на рефакторинг)
- Перед введением код-ревью договоритесть о код-стайле, желательно вообще подключить литер
- Помните, что ревью - важная часть процесса, чувствуйте свою ответственность: работайте над ревью в несколько этапов - сначала просмотрите "по диагонали", затем погрузитесь в логику и код - это позволит максимально быстро дать обратную связь в случае каких-либо архитектурных ошибок
- Всегда помните о решаемой задаче и сроках. Убедитесь, что код решает задачу и не переусложнен. Иногда совершенствование логики лучше вынести в дополнительные задачи
- Будьте вежливы. Замечания пишите обезличенно, без обращения к кому-то конкретному
- Сразу договоритесь, кто имеет финальное слово или право вето на любой вопрос в код-ревью. Это будет полезно при возникновении нерешаемых противоречий между членами команды
- И еще раз - ревью это часть большого процесса, поэтому следите за тем, чтобы задача не "зависала" на этой стадии
