если нет доверия тимлиду - надо его менять

тимлид это прежде всего человек, у него сейчас все хорошо и он выдает результат, завтра у него жена ушла с любовником и все скатилось в говно,а корявых решений он уже тебе наплодил, и кто потом будет это расхлебывать когда оно в бой уйдет?

кросс-командое ревью предполагает, что у нас несколько команд схожей квалификации, но занимающихся разными задачами. А не "маркетологи оценивают врачей"

результаты выдаваемые человеком не одинаковы день ото дня, он сегодня пишет волшебный код а завтра какую то херню творит

такое только на галере возможно

нет, в любой крупной ИТ-компании, где ИТ-отдел больше 5-7 человек

в результате только еще больше ненависти

это зависит от организации кросс-ревью. Это задача менеджмента сделать так, чтобы ревью не воспринимался как репрессивная мера

потому что ревью делать тоже надо уметь, некоторые пишут на ревью "это хуйня, переделай", а некоторые укажут что будет работать не оптимально, подскажут как можно лучше и дадут ссылки

а толку от мозгоштурма?

все много говорят, а проблема остается.

ревью нужно чтоб там не пролезли косяки которые автору могут быть не видны, потому что он этот код писал

зачем ревью делают

вы явно не понимаете, что такое ревью и для чего оно нужно. Но для России это нормально :)

на ревью часто вылезают проблемы когда один и тот же код пишется дважды, так как автор например не знает старые подсистемы

при чем тут ревью

на ревью часто вылезают проблемы когда один и тот же код пишется дважды, так как автор например не знает старые подсистемы

потому что ревью делать тоже надо уметь, некоторые пишут на ревью "это хуйня, переделай", а некоторые укажут что будет работать не оптимально, подскажут как можно лучше и дадут ссылки

В результате люди со знаниями, которые могли бы комитить код и решать свои задачи тратят время на чужой прогресс. Вообще любое ревью возможно, только если ревьюер заинтересован в результате всей команды. А значит он точно должен быть из той же команды и желательно быть лидом на проекте. Все остальные ревью если и существуют, то просто для галочки, других примеров живьем не видел и ее слышал , кроме как со сцен конференций,но там всегда розовые пони.

В результате люди со знаниями, которые могли бы комитить код и решать свои задачи тратят время на чужой прогресс. Вообще любое ревью возможно, только если ревьюер заинтересован в результате всей команды. А значит он точно должен быть из той же команды и желательно быть лидом на проекте. Все остальные ревью если и существуют, то просто для галочки, других примеров живьем не видел и ее слышал , кроме как со сцен конференций,но там всегда розовые пони.

люди со знаниями которые проводят ревью и так делают вклад, они улучшают качество кода проекта, повышают уровень команды, это тоже очень важно

они во-первых читают более качественный код, чем могут сами написать (иногда) во-вторых, въезжают как все устроено в-третьих задают правильные вопросы, если что-то непонятно

да, я согласен что младшие тоже должны ревьюить, и лучше чтоб если нет других старших чтоб проревьюил младший чем вообще никто не проревьюил

во втором случае никакая методология не нужна даже даром.

да, я согласен что младшие тоже должны ревьюить, и лучше чтоб если нет других старших чтоб проревьюил младший чем вообще никто не проревьюил

они во-первых читают более качественный код, чем могут сами написать (иногда) во-вторых, въезжают как все устроено в-третьих задают правильные вопросы, если что-то непонятно

Согласен, это тоже видел живьем, но это способ обучения, а не способ улучшения качества продукта (в краткосрочной перспективе) откуда изначально обсуждение надобности ревью выросло

у нас код ревью делают программисты в рамках одной команды и иногда из других команд, по ревью админов - щас у них вообще нет ревью, но вроде как договорились что программисты тоже будут смотреть что там происходит

да. но я еще видел что компании, у которых в приоритете стоят краткосрочные перспективы обычно плохо заканчивают.

Вырастить мидлов из джуниоров благородная цель, но они всегда сваливают чуть летать научатся :) вот прям совсем всегда

Вырастить мидлов из джуниоров благородная цель, но они всегда сваливают чуть летать научатся :) вот прям совсем всегда

ревьювить погромистов и ревьювить админов сильно разные задачи.

а чтобы админ ревьюил программера, надо ему погрузиться в бизнес-логику и архитектуру

все много говорят, а проблема остается.