M
Если команда и чел наворотил 5к без чек ревьи по дороге это плохо
Size: a a a
2020 August 11
AZ
А зачем такое вообще ревьювить? Его можно несколько дней ревьювить лол
а что с ним делать?
p
Нет ну в опен сорсе
Ну у нас такие ПРы обсуждаются в телоксид чатике поэтому ревьюв редко нужен, тупо в мастер хуячим и норм
AZ
Если команда и чел наворотил 5к без чек ревьи по дороге это плохо
ревьювится задача готовая. Зачем смотреть рыбу-мясо? У нас как бы разработчики нормальные, за ними пригляд не нужен
CD
А зачем такое вообще ревьювить? Его можно несколько дней ревьювить лол
Потому что тебе в любом случае надо понимать, что в кодовой базе происходит
p
ревьювится задача готовая. Зачем смотреть рыбу-мясо? У нас как бы разработчики нормальные, за ними пригляд не нужен
Зачем тогда ревьюв? :)
AZ
Зачем тогда ревьюв? :)
чтобы исправить косяки
AZ
архитектуру продумыают до того как задача в работу берется
CD
А зачем такое вообще ревьювить? Его можно несколько дней ревьювить лол
Ты иначе не можешь нарабатывать опыт использования тех или иных либ и так далее. Ты должен знать, где сложности, где проблемы с архитектурой, где что выносить. Как минимум просматривать для этого ты должен все правки
M
ревьювится задача готовая. Зачем смотреть рыбу-мясо? У нас как бы разработчики нормальные, за ними пригляд не нужен
Ну хз мне кажется 5к это перебор так можно и до 20к доползти ну а чё фитча же большая
M
Всеравно какой то верхний предел должен бытт
p
ревьювится задача готовая. Зачем смотреть рыбу-мясо? У нас как бы разработчики нормальные, за ними пригляд не нужен
Эта тема обсасывалась в чате тимлидов не одну сотню раз. Пришли к выводу что ревьюв нужен только джунов. Потому что тратить 5-10% времени на тщательный ревьюв не хочется, а нетщательный редко находит баги.
CD
архитектуру продумыают до того как задача в работу берется
ага, а потом от исполнителя "бля мы херово тут напроектировали" :)
M
Эта тема обсасывалась в чате тимлидов не одну сотню раз. Пришли к выводу что ревьюв нужен только джунов. Потому что тратить 5-10% времени на тщательный ревьюв не хочется, а нетщательный редко находит баги.
Ну это скорей даже не для поиска багов
M
Вот представь кодовая база шагнула за два лимона
M
Понятно дело что то ты можешь накостылять или криво заюзать
M
Вот для этого ревью что бы потом не нефакторить много
CD
Эта тема обсасывалась в чате тимлидов не одну сотню раз. Пришли к выводу что ревьюв нужен только джунов. Потому что тратить 5-10% времени на тщательный ревьюв не хочется, а нетщательный редко находит баги.
Можно ссылочки? В частности, там кто-нибудь изучал, как ревью влияет на формирование единого стиля программирования в команде?
M
Ну + таки баги тупые часто бывают пролетают
M
Я скока ревьювил часто находил то забытый возврат ошибки то миснутый чек на что то