Как проводить код-ревью полезно, а не по вкусу?
- На что смотреть при код-ревью?
- Как писать комментарии на ревью?
Смотреть по существу в порядке важности: правильность (решает ли задачу, крайние случаи, ошибки), безопасность (ввод, права, секреты), данные и совместимость (миграции, API), тесты, поддерживаемость (понятность, сложность, дублирование). Стиль и форматирование — на линтер, а не на человека. Комментарии — конкретные, с причиной и вариантом, с пометкой важности («блокирует», «можно позже», «мелочь»). Отвечать быстро: долгое ревью тормозит всю команду.
- Вопрос вместо приказа: «что будет, если список пустой?»
- Отмечать и хорошее — ревью тоже обучение
- Большой PR — попросить разбить, а не читать поверхностно
А ещё Ревью проверяет код, а не человека: «здесь гонка», а не «ты не подумал»
Подробно
Чек-лист ревьюера:
- понятно ли, что меняется и зачем;
- корректность и обработка ошибок;
- безопасность и права доступа;
- производительность там, где важна (запросы в цикле);
- тесты на новое поведение;
- не ломает ли совместимость (API, схема БД, конфиг).
Формулировки:
- «Блокирует: при пустом items будет IndexError на строке 42 — добавить проверку?»
- «Мелочь: можно назвать total_amount — сейчас неочевидно».
Скорость: договорённость в команде, например ответ в течение рабочего дня.
Что хотят услышать
- Приоритеты: правильность, безопасность, совместимость, тесты.
- Стиль на линтер.
- Конкретные комментарии с уровнем важности.
- Скорость и уважительный тон.
Проверьте себя
Один верный вариант. Подумайте, потом откройте ответ.
Показать варианты и ответ
- ✗Главная задача ревью — привести код к единому стилю автора ревью: форматирование, имена, порядок импортов; логику и тесты проверяет QA
- ✓Смотреть правильность, безопасность, совместимость и тесты; стиль — на линтер; комментарии конкретные, с причиной и уровнем важности
- ✗Большой PR достаточно пролистать и одобрить
- ✗Хорошее ревью — это много комментариев о форматировании
Разбор темы
- Pull request, код-ревью и стратегии ветвления
Как оформить pull request, который быстро проверят, как ревьюить по существу, чем trunk-based отличается от GitFlow и что делает CI/CD.
Связанные вопросы
Разбор собеседования с sotto
После собеседования sotto разбирает разговор: оценка, каждый вопрос — хорошо, частично или слабо, чего не хватило в ответе и над чем поработать к следующему разу. Первые 30 минут бесплатно.