Ответ
Да, код ревью — обязательная часть нашего процесса разработки. Каждый пул-реквест проходит проверку минимум одним коллегой перед мержем.
Правила:
- Проверяем не только функционал, но и архитектуру, читаемость, соблюдение гайдлайнов
- Комментарии должны быть конструктивными с предложениями по улучшению
- Используем GitHub/GitLab ревью-инструменты
Пример:
// До ревью
fun calculate(items: List<Item>): Int = items.sumBy { it.price }
// После ревью (учтены edge cases)
fun calculateTotalPrice(items: List<Item>?): Int {
return items?.sumOf { it.price } ?: 0
}
Ревью помогает находить баги на ранних этапах и делиться знаниями в команде.