Ответ
В команде мы проводили код-ревью через GitHub/GitLab PR или Gerrit. Основные принципы:
- Обязательность: Без апрува хотя бы одного коллеги мержить нельзя.
- Четкие критерии: Проверяем архитектуру, читаемость, тесты, перфоманс.
- Автоматизация: Используем Detekt, ktlint, SonarQube для статического анализа.
- Время на ревью: PR не должен висеть дольше 1-2 дней.
Пример комментария в PR:
// Было
fun getUser(id: Int) = repo.getUser(id)
// Ревьюер:
// Добавь обработку null, если пользователь не найден:
fun getUser(id: Int): User? = repo.getUser(id) Важно: ревью — не критика, а совместное улучшение кода.