Как правильно делать код-ревью? Часть 2: навигация по ревью, скорость, комментарии, конфликты

[reforms]

Понять, нужен ли представленный функционал в принципе и есть ли у него хорошее описание

Зачем вообще ставить задачу на функциональность, которая не нужна?

Обратить внимание на самое важное в измененном коде и то, насколько хорошо решение спроектировано в целом

К сожалению, не все это понимают, для многих стоп-фактором может быть сущий пустяк.

Скорость проведения ревью: отвечайте быстро

Мне кажется это самый важный пункт из статьи, если не в ущерб качеству. И чем быстрее будет получен ответ тем лучше.

[JackMonterey]

Зачем вообще ставить задачу на функциональность, которая не нужна?

Актуально для open source проектов, где пул-реквест может представить любой, а не только члены закрытой команды разработчиков.