Она уже решена — на этих встречах вскрылись проблемы, о которых раньше было лень говорить. Например, те же медленные ревью

Я давно задаю этот вопрос. Но все конечно осознают важность ревью. Но почему-то повсеместно считают его второстепенной активностью, которая может ждать день-два-неделю

А какие-нибудь альтернативы рассматривали для той же цели (наверное, качество кода и стиля)?

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

Никаких не рассматривали. Каждый раз когда я поднимал этот вопрос в чатиках, начиналось «да, ревью надо делать, давайте обратим на это внимание и будем делать это чаще» и никаких других действий. За все хорошее, против всего плохого, короче. А результата нет, кто-то раньше как делал ревью по сто лет, так и делает, особенно это касается тех, у кого экспертизы больше

Ну вот подобная штука у меня была, пока был человек, который форсил code review, регламент был, каждый день час на CR тратился и всё такое. Не всем это нравилось, но сидели и молчали. Потом человек ушёл и опять забили на CR, и он опять мешать стал. Пробовали ещё 2 варианта - оптимистичный CR (когда те кому это нравится - реально этим занимаются и у вас есть рефакторинг в процессе), и прям экстремальный - парное программирование, по сути непрерывное code review. Можете такие подходы занести, обсудить, рассмотреть, поэкспериментировать. Главное, изначально цель этой штуки не забыть.

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

ПП похоже совсем не знают или не хотят практиковать, я предлагал и мне сказали «попробуем, когда задача появится». Позитивное ревью это интересно — выделить список людей, которые дают SLA на кодревью

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

В смысле если его не посмотрели в течение какого-то времени, то закрываем молча без ревью?

Звучит как форма забастовки))

Технически, у вас на PR нельзя накинуть комментарии, если код уже слит в ветку релиза?

У нас монорепа и транк бейзд девелопмент. Если код в транке, то поди его откати, жопу оторвет (простите)

Ну фиче тоглы ж есть, наверное?) Если совсем дичь какая влетит

Короче после коммита уже кулаками не машут, если это не ломающее изменение

Вообще, откатывать не надо, надо рефачить

Есть

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

Но эта штука требует тестов

Иначе рефакторинг будет следущей практикой, которая всех будет убивать