Как было организовано code review на прошлой работе

В команде использовали GitHub Pull Requests. Каждый PR должен был пройти ревью хотя бы одного коллеги перед мержем.

Основные правила:

• PR не должен превышать 300-400 строк
• Обязательные тесты для нового кода
• Комментарии в коде на английском
• Описание изменений в PR

Пример workflow:

1. Создаем feature-ветку от main
2. Делаем изменения, пушим
3. Создаем PR, назначаем ревьюверов
4. После апрува - мержим через squash commit

Особое внимание уделяли:

• Соответствию кодстайлу (ESLint)
• Производительности (avoid N+1 queries)
• Доступности (aria-атрибуты)
• Чистоте кода (DRY, KISS)