Как проводить кодревью?

[YourgenAP]

Проблемы увидел. Решения не увидел. Кто поможет?

[berlicon]

Чеклисты, гайдлайны, тулзы для стат анализа, бот распределяющий MR по разрабам и все перечисленное в статье и в последних 2х пунктах опроса не зашло? ?

[vadimkle]

тулзы для стат анализа - это то, чем надо пользоваться в процесса review? Довольно странно, на мой взгляд.

[berlicon]

Если большой MR имеет смысл смотреть его не в гитлабе, а залить себе ветку локально, запустить линтер и выгрузить в результаты ревью список предупреждений линтера, связанных с измененным кодом.

[idmx]

Было бы удобнее, если бы столбцы в таблицах были бы хоть как-то подписаны. Сложно сразу сориентироваться. Читал с телефона, сначала показалось, как бессвязный текст в столбцах

[berlicon]

Так там нечего подписывать, в таблицах - это пункты, описывающие главу, например, глава "Паттерны" и в таблице ниже - список паттернов.

[berlicon]

Выложил последнюю версии презентации: https://docs.google.com/presentation/d/1FvGtsMSbF9-cpO6vATe1HxQgCBTS293phbaf9vKSrio/edit?usp=sharing