Как устроен Code Review в последнем проекте

В нашем проекте Code Review проходит через GitHub Pull Requests. Основные правила:

1. Перед созданием PR запускаются все тесты и статический анализ (Checkstyle, SpotBugs)
2. Обязательно 2 аппрува от коллег
3. Проверяем:
   • Соответствие код-стайлу
   • Качество тестов
   • Возможные уязвимости
   • Оптимальность алгоритмов

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

// Вместо этого:
for (int i = 0; i < list.size(); i++) {
    process(list.get(i));
}

// Лучше использовать for-each:
for (Item item : list) {
    process(item);
}

Автоматизированные проверки через GitHub Actions включают сборку, тесты и анализ кода. Среднее время ревью - 1 рабочий день.