Ревью

Ревьювер - человек, который осматривает код.

Ревью - поверхностный осмотр кода. Поверхностным осмотр является потому, что ревьювер не несет 100% ответственности за код.

Все ревью стараемся проводить максимум в 2 итерации! Если ревью в 2 итерации не укладывается - разработчик с ревьюером пытаются совместно разобраться в проблеме.

Как задача попадает в ревью

Когда по задаче написан код, разработчик в задаче:

1. указывает в поле Статус: "Требует ревью"
2. указывает в поле Исполнитель: "Веб - разработчики".

*заметка: чтобы тестировщик мог проверить задачу, в задаче должно быть заполнено поле: merge request

Как задача попадает к ревьюверу

Ревьюер для поиска задач требующих ревью используется фильтр "Требует ревью". В списке выбирает задачу сверху и переводит ее в статус "Проходит ревью", а в поле назначена указывает себя. Для этого можно воспользоваться кнопкой "Начать ревью"

Что если

• Если задача не прошла ревью, она отправляется на исправление - статус "Ожидается разработка", в поле Исполнитель указывается разработчик выполнявший задачу.
• Если задача прошла ревью, она отправляется в тестирование - статус "Ревью завершено", в поле Исполнитель: "Веб - тестировщики". Для этого можно использовать кнопку "Ревью пройдено".

Заметки

Подходы осмотра - чем больше кода, тем больше замечаний к коду, а значит тем больше подходов осмотра.

Чем больше количество разработчиков выполняют осмотр, тем больше надежд разработчик возлагает на ревьюверов, поэтому ревьювер всегда 1, максимум 2.

Ухудшение осмотра

Замечено, что качество осмотра падает в следующих случаях:

1. ухудшение осмотра прогрессирует от первого к последнему подходу осмотра (самое тщательное ревью - первое, самое поверхностное - последнее), интересно то, что увеличение кол-ва строк кода никак не влияет (не в лучшую сторону, ни в худшую)
2. чем меньше времени ревьювер готов уделить на ревью, тем хуже осмотр (ревьювер может быть загружен своими задачами)

Источники: 1