По последнему посту, связанному с ревью, подписчик прислал дополнения, которыми с радостью делюсь.
Есть еще такое ревью, которое проводят члены команды и решение принимается без тимлида.
Недостатки, помимо психологического фактора:
• время. Пулл реквесты могут накапливаться и есть риск не влезть в релиз из-за накопившегося ревью.
• зависимости. Если пуллреквест один, то терпимо, но если надо работать только после того, как будут смерджены чужие изменения (один код или PR в общую библиотеку), то ты будешь заблокирован до тех пор, пока не закончится кодревью.
• возможность удобной работы с инструментами CI: можно еще на уровне пулл реквеста прогнать все тесты, собрать целиком приложение, пройтись всеми линтами до того, как изменение попадет в мастер.
Также теоретически возможно инспекционное тестирование: когда инженер QA после ревью и прохождения СI-тестов сначала тестирует изменение, забирая его и собирая приложение с новым изменением, а потом уже позволяют мерджить, если тестирование прошло успешно.
P.S. Приятно, когда вы даете обратную связь по постам и присылаете #вопросы. Всегда на связи тут.
Post #182
2.9K