A
в общем
code review, как и open space)))
просто надо правильно научиться использовать и никаких неудобств не будет)))
code review, как и open space)))
просто надо правильно научиться использовать и никаких неудобств не будет)))
Size: a a a
2020 October 13
OS
Чтобы была команда внедряйте CI. В таком случае разработчики будут получать изменения друг друга по несколько раз в день, достаточно маленькими проциями.
Кажется CI не для этого:) А разработчики и так делают git pull вручную:)
AK
Если у вас есть CodeReview (а значит долгоживущие ветки), у вас отсутсвует CI. По определению 🤷♂️
DP
а?!
OS
Прости, но нет:)
A
Если у вас есть CodeReview (а значит долгоживущие ветки), у вас отсутсвует CI. По определению 🤷♂️
хмм) это почему это codereview как то связан с временем жизни ветки?)
AK
Прости но да. 🙂
OS
code review никак не кареллирует с долгоживущими ветками:)
A
вообще то review нужно на каждый коммит писать
AK
code review никак не кареллирует с долгоживущими ветками:)
Он может быть Branch Based и Commit Based. В случае с CI он Commit Based.
OS
вообще то review нужно на каждый коммит писать
С этим тезисом тоже не соглашусь:)
A
и вообще-то единственный момент который советуют не перегружать того кто делает review
большими коммитами)) то есть вот практически CI сразу))
большими коммитами)) то есть вот практически CI сразу))
A
С этим тезисом тоже не соглашусь:)
хорошо)
а почему?)
а почему?)
Н
Если у вас есть CodeReview (а значит долгоживущие ветки), у вас отсутсвует CI. По определению 🤷♂️
Как это связано?
OS
По крайней мере с формулировкой:) Иногда одна задача это несколько отдельных коммитов. Задача ревьювится целиком. Конечно можно попробовать отдать коммиты разным людям...
DP
коллеги, а есть смысл ревьювить что-то меньше PR?
Н
По крайней мере с формулировкой:) Иногда одна задача это несколько отдельных коммитов. Задача ревьювится целиком. Конечно можно попробовать отдать коммиты разным людям...
+, feature branch, то есть вся задача на ревью.
A
По крайней мере с формулировкой:) Иногда одна задача это несколько отдельных коммитов. Задача ревьювится целиком. Конечно можно попробовать отдать коммиты разным людям...
я имел ввиду то что каждый коммит желательно чтобы было review )))
а распределение по тому сколько и кому - это уже другой момент)
а распределение по тому сколько и кому - это уже другой момент)
Н
Зачем ревьювать каждый коммит, если работа над задачей в прогрессе?
OS
Вот меня твоя формулировка немного смущает, Alex:)