VF
А каков средний размер коммита? Ну или от до
Size: a a a
2021 May 08
VF
А, ну тогда все понятно )) если все знают чего должно быть в конце, то вопросов нет
OS
Сторонники кодревью:
- кодревью помогает учить людей за год.
Тоже сторонники кодревью:
- тратим на это 10 минут в день.
- кодревью помогает учить людей за год.
Тоже сторонники кодревью:
- тратим на это 10 минут в день.
VR
да разные размеры. бывают и на 1000 изменений — но это зло )
VR
но от код-ревью мы не готовы отказаться — это хорошо работающий инструмент, который делает продукты более стабильными
V
стабильными в смысле багов меньше?
V
а можете пожалуйста посмотреть, сколько во время код ревью было найдено багов за последний месяц?
VF
Ну когда у всех будет кодревью занимать 10 минут и не откладывать задачу в долгий ящик, то я не буду выдвигать лозунгов
OS
Если кто-то не готов отказаться - не надо их торопить. Пусть готовятся дальше.
VR
если в мастер не упал багованный код, то бага нет ) что мерять то? )
VR
таки к чему готовиться? 🤔
V
ну я так полагаю, что если в рамках код-ревью были найдены баги, то фичу вы наверное отправили на доработку с пометкой “тут баг вот” ?
OS
К освоению других, более эффективных практик
VR
ммм, ревью идёт не на фичу, а на МР — исправляется на месте
V
ну так там пометочка-то наверное есть всё же человеку в MR — "вот тут баг”?
LT
Второй день про код ревью? Как вас всколыхнуло 😅
VR
как из двух практик "проводить код-ревью" и "не проводить код-ревью" — вторая может быть эффективнее?
VR
нет, что Вы, это неконструктивная пометочка — должно быть конкретно написано, все консёрны сразу
VR
целый день читал, чтобы ответить ))