Выбор ответа Middle Код-ревью обновлено
С чего начинать ревью
Вопрос
В каком порядке эффективнее всего смотреть пулреквест?
Попробуйте сами
Выберите вариант, и откроется разбор.
Показать ответ и разбор
Коротко
Правильный ответ: Корректность и задача, затем безопасность и архитектура, стиль линтеру. Сначала важное: решает ли PR задачу, нет ли багов и уязвимостей, правильно ли встроен в архитектуру.
Варианты ответа
- Стиль и отступы, потом имена переменных, потом логика и архитектура
- Корректность и задача, затем безопасность и архитектура, стиль линтеруправильный ответ
- Только тесты: если CI зелёный, код корректен и можно апрувить
- Сначала проверить, что код написан в твоём стиле, потом всё остальное
Разбор
Сначала важное: решает ли PR задачу, нет ли багов и уязвимостей, правильно ли встроен в архитектуру. Мелочи стиля должны ловить линтер и форматтер, человек на них только тратит внимание. Зелёные тесты не гарантируют корректность, если тесты не покрывают краевые случаи. Личные вкусы не повод для блокирующих замечаний.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.