Комментарии 17
В данном контексте думаю скорее нет, чем да. Код ревью это процесс асинхронный. Смысл в том что помимо основной очереди из тасков на выполнение у девелопера есть еще очередь на ревью. Пытаться синхронизировать выполнение ревью с задачей менторинга новичка выходит за рамки контекста. Вы пытаетесь притянуть частный случай когда исполнитель и ревьювер являются одновременно ментором и учеником и оба располагают достаточной свободой в расписании. Но в динамичных и больших командах это в общем случае не так. Исполнитель и ревьювер могут не знать друг друга лично, могут находиться в разных частях планеты и обычно обладают примерно одинаковой квалификацией.
То есть конечно можно совмещать менторинг и онбоардинг с кодревью, но в общем случае это две разные задачи.
То есть конечно можно совмещать менторинг и онбоардинг с кодревью, но в общем случае это две разные задачи.
Для этого нет необходимости сидеть над ревью вместе.
Если общий стиль уже выработан — вырабатывать его повторно нет большого смысла. А тем более отвлекать всю команду на то, что бы просмотреть изменения по всем задачам.
Ну и да — если у вас изменений за релиз можно разобрать за один час, то это очень немного изменений )
Для каждого конкретного человека, которого ревьюят это так же полезно как нормальный ответ ревьюэра в задаче.
А парное программирование — это вообще очень специфическое занятие никак сюда не относящееся… и вообще сомнительное. =)
Ну и да — если у вас изменений за релиз можно разобрать за один час, то это очень немного изменений )
Для каждого конкретного человека, которого ревьюят это так же полезно как нормальный ответ ревьюэра в задаче.
А парное программирование — это вообще очень специфическое занятие никак сюда не относящееся… и вообще сомнительное. =)
Парное программирование в моём опыте это не совместное решение общей проблемы программистами одного уровня, а метод обучения.
На мой взгляд иногда это совместное решение проблем, чаще менторство, ну а иногда способ трансферить знания и умения между разработчиками одного "уровня".
И если время и начальство позволяют, то по мне это вполне себе полезная штука:)
Парное программирование в моём опыте это не совместное решение общей проблемы программистами одного уровня, а метод обучения.Эффективность спорная, конечно. Ну да — методы каждый выбирает сам ) Как и то — работать ли вообще с новичками которых надо обучать.
Не много, но проблема в том что с той стороны сидят люди из Бангалора и если раз десять им одно и тоже не покажешь, то толку не будет. Хотя и после десяти раз постоянно косячат на ровном месте, где казалось бы уже негде.Ну, тугим людям, мне кажется, намного эффективнее персонально возвращать задачу с пояснениями, чем смотреть на отсутствие мысли в их глазах на общих собраниях (обычно, на них всё пролетает мимо ушей). Но господам из Бангалора я ничего не обяснял, может есть кейсы в которых это и эффективно. У меня не получалось такого (
Не много, но проблема в том что с той стороны сидят люди из Бангалора и если раз десять им одно и тоже не покажешь, то толку не будет.
Можно внедрить использование Каталога Рефакторинга. Мы таким образом многократно сократили время на Code Review. Правда, сегодня этот каталог сильно изменился, и уже не так удобен в использовании как прежде (по моему собъективному мнению). Прелесть каталога в том, что не нужно тратить время на объяснения — за вас будет работать соответствующий раздел книги, на который можно пройти прямо со страницы каталога.
А парное программирование — это вообще очень специфическое занятие никак сюда не относящееся… и вообще сомнительное. =)
Парное программирование — это частный случай Continuous Review. Если Вы про Review, то таки относится, по крайней мере — формально…
Интерестно, а бывают рассовые конфликты и как их разрешают
Типа жалоба начальству — ты слишком строго ревьюишь мой код, потому-что я другого пола, другой рассы, другой веры…
Типа жалоба начальству — ты слишком строго ревьюишь мой код, потому-что я другого пола, другой рассы, другой веры…
What the hell is «CL»?
Везде в документации встречается эта аббревиатура, но нигде не расшифровывается. Change Log? Commit Log? Что именно имеется в виду, кто подскажет?
Везде в документации встречается эта аббревиатура, но нигде не расшифровывается. Change Log? Commit Log? Что именно имеется в виду, кто подскажет?
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Как проводить код-ревью