Как проходило Code Review на последнем проекте

На последнем проекте Code Review было обязательным этапом перед мержем в основную ветку. Мы использовали GitHub Pull Requests с обязательным approval минимум одного коллеги.

Процесс:

1. Разработчик создает PR, добавляет описание изменений
2. Автоматически запускались линтеры и тесты (через GitHub Actions)
3. Ревьюверы проверяли:
   • Соответствие гайдлайнам
   • Качество кода (читаемость, производительность)
   • Наличие тестов
   • Корректность типов (TypeScript)

Пример комментария:

// Было
const data = response.data.map(item => ({...item, id: item._id}))

// Ревью: лучше избегать лишних копирований
const data = response.data.map(({_id, ...rest}) => ({...rest, id: _id}))

Обсуждения велись в комментариях PR. Важные правки вносились сразу, спорные моменты обсуждались в Slack. В среднем PR проходил ревью за 1-2 дня.