делаете? Какие вещи должен посмотреть reviewer?
1) представь, что тебе через год надо допилить что-то в этом куске кода, и с учетом этого уже смотри, что будет вызывать маты 2) проверка неявных бизнес-зависимостей/ошибок — эт если ты к данному куску системы непосредственно не относишься, но можешь его дергать 3) кодстайл и очевидные баги, если ленивый
все, до чего можешь докопаться, в зависимости от отношения к тому, чей код ревьювишь (начиная от ошибок в бизнес-логике и заканчивая формулировками комментариев в коде)
все от реализации логики до покрытия ее тестами
Обсуждают сегодня