AK
коллеги, а есть смысл ревьювить что-то меньше PR?
Если у вас CI ревьювить имеет смысл только коммиты которые упали. Если нет CI то тогда PR
Size: a a a
2020 October 13
OS
Человек тратит X времени, делает сколько угодно коммитов, слияний, поглощений... когда этот человек считает что работу он выполнил или когда считает что ему нужна помощь извне... тогда code review всей задачи.
A
Зачем ревьювать каждый коммит, если работа над задачей в прогрессе?
например, чтобы коллеги знали что ожидать в скором времени и то что работа в нужном направлении?)
OS
Если у вас CI ревьювить имеет смысл только коммиты которые упали. Если нет CI то тогда PR
Вот у меня есть CI... и есть N репозиториев и в каждом по X веток... и я в свою ветку пушу на сервер... и на CI запускаются тесты - мне, простите, какое ревью нужно?:)
Н
например, чтобы коллеги знали что ожидать в скором времени и то что работа в нужном направлении?)
У коллег есть работа помимо ревью кода, который в конце работы над задачей может быть другим
A
Человек тратит X времени, делает сколько угодно коммитов, слияний, поглощений... когда этот человек считает что работу он выполнил или когда считает что ему нужна помощь извне... тогда code review всей задачи.
да) кстати)))
у меня были неточности)))
ревью кода в "production branch" скажем так))
я естественно не предлагал что вообще все коммиты куда-угодно)))
у меня были неточности)))
ревью кода в "production branch" скажем так))
я естественно не предлагал что вообще все коммиты куда-угодно)))
AK
Вот у меня есть CI... и есть N репозиториев и в каждом по X веток... и я в свою ветку пушу на сервер... и на CI запускаются тесты - мне, простите, какое ревью нужно?:)
Если у вас есть feature-branch которые живут дольше одного дня - у вас нет CI
OS
да) кстати)))
у меня были неточности)))
ревью кода в "production branch" скажем так))
я естественно не предлагал что вообще все коммиты куда-угодно)))
у меня были неточности)))
ревью кода в "production branch" скажем так))
я естественно не предлагал что вообще все коммиты куда-угодно)))
Вот это-то и смущало:) Разобрались:)
@
Человек тратит X времени, делает сколько угодно коммитов, слияний, поглощений... когда этот человек считает что работу он выполнил или когда считает что ему нужна помощь извне... тогда code review всей задачи.
я помню как у лида команды бомбануло, когда ему понадобилось 5000+строк кода ревьювить.
OS
Если у вас есть feature-branch которые живут дольше одного дня - у вас нет CI
Эм...
A
Вот это-то и смущало:) Разобрались:)
ну вот)))
даже здесь ревью полезно))
потому что то что иногда "подразумеваешь"))
не всегда сразу удаёться правильно написать)))
даже здесь ревью полезно))
потому что то что иногда "подразумеваешь"))
не всегда сразу удаёться правильно написать)))
OS
я помню как у лида команды бомбануло, когда ему понадобилось 5000+строк кода ревьювить.
Это плохо, но иногда это нужно:)
A
откуда мораль))) каждому senior по code review от тимлида - и будет счастье)))
OS
Особенно мне нравятся ревью где правки по нему не обязательны:) И тогда люди получая от тебя портянку правок... просто их все игнорят, говоря что если они будут всё это править то проект никогда не выйдет... ну ок... зачем тогда тебе ревью, пиши дальше так:)
A
я помню как у лида команды бомбануло, когда ему понадобилось 5000+строк кода ревьювить.
обычно в таких случаях того кто так пошутил над тимлидом, отправляют писать тесты с 200% покрытием кода))
OS
откуда мораль))) каждому senior по code review от тимлида - и будет счастье)))
Ревьювер может быть рандомным, более того это предпочтительнее.
A
Ревьювер может быть рандомным, более того это предпочтительнее.
мне кажеться меньше 100% покрытия чем либо - это всё же от недостатка ресурсов)
OS
Понятие 100% по количеству строк кода или по смысловой нагрузке?:)
AK
Эм...
Ну да. Термину CI уже 29 лет, изучите вопрос. Он появился до того как появился GitHub, до того, как программы для автоматизации процессор разработки переименовались в CI/CD системы и до того как появился Git 🙂
A
Понятие 100% по количеству строк кода или по смысловой нагрузке?:)
как говорят, в реалиях жизни - "хоть код посмотрите" уже хорошо)))