чем дженкинс не угодил? ну давайте на ТС или Бамбу поменяем) не держите в себе)
Да, запускать в голове код. Смысл ревью не только в том, что бы соответствие стайлгайдам проверить. Одна из целей - шаринг знаний, которая не возможна без понимания как этот код будет работать. Вторая - проверка корректности имплементации, с точки зрения логической структуры и паттернов. Ну и третья про то, что если ты не можешь в ходе ревью сказать, что делает этот код - значит код говно.
Обсуждают сегодня