СС
В целом ревью неплохой инструмент, особенно для калибровки новичков в команду.
Но нужно понимать, какую ты проблему хочешь решить с помощью код-ревью.
Но нужно понимать, какую ты проблему хочешь решить с помощью код-ревью.
Size: a a a
2021 June 01
АГ
так то в целом обычно понятно про что это - но сильно вникать... хз чот. раз человек сделал, проверил, и это работает - то смысл туда вникать глубоко, вчитываться и т д
АГ
ну можно конечно какие-то явные косяки найти
A
Я думаю такой тип ревью не должен быть сфокусирован на решении и архитектуре.
Основное - это снижение бас фактора, дизайн ревью, нейминг, качество документации, реализация тестов, обмен знаниями
Основное - это снижение бас фактора, дизайн ревью, нейминг, качество документации, реализация тестов, обмен знаниями
АГ
когда какие-то инструменты не правильно применяются, или тестов нет, или сделано то что уже сделано, или еще что-нибудь. но чаще всего системы сложные, запутанные, в них никто не шарит, и все это в принципе бесполезно. все равно пропускаешь все косяки и они потом вылезают
АГ
а документация как к ревью относится?
АГ
у вас к каждому методу большая портянка с комментом? инфраструктурные штуки разрабатываете? опенсорс?
АГ
в опенсорсе да, ревью обязательная вещь
A
Нет конечно, но мне кажется достаточно важно, чтобы по доке было понятно о чем сервис и как он работает
A
И что-то от опенсорса есть: можешь прийти в сервис, который мейнтейнит другая команда, законтрибьютить, получить апрув и это потом катится в прод
АГ
а с какого перепуга это делать?
A
Автономность
АГ
если ты используешь его, и там чего-то не хватает - то это понятно. задачи какие у вас?
A
Именно такие и задачи. У нас над продуктом работают автономные команды, поэтому часто есть зависимости, общие компоненты и т.д
АГ
какие же они автономные если есть зависимости)
АГ
я бы сказал что в такой ситуации у вас часто одно и то же сделано несколько раз чуть разными способами.
АГ
причем это трудно выявить, потому что надо вникать. хорошо если нейминг схожий, а может быть и не схожий
A
Автономные != независимые. Если какая-то команда реализовала что-то, что ты можешь тоже использовать, то зачем форкать и дублировать?
АГ
должен быть какой-то общий чат где подобными наработками делятся
АГ
и спрашивают, "а не решали ли вы уже такую задачу"