Интересно, новичку дают задачу, которую он пилит в итоге полтора месяца, попутно получая сотни комментариев. Куча производственных потерь, человеку сразу дают понять "кто он тут" - кажется сode review тут не самая большая проблема. Через полтора месяца во многих компаниях эта задача уже никому не нужна, видимо и новичка нанимали зря. Элементарный Скрам бы сразу подсветил эту проблему и настучал по голове всей команде.

Данное видео не найти ни среди видео канала Podlodka Podcast ни среди подкастов в Castbox, а жаль: обсуждение получилось очень интересным.

@OlegSoroka

4 жыл бұрын

Это часть платного контента, но ради пользы индустрии - пошарено :)

@grysfeek

4 жыл бұрын

@@OlegSoroka сегодня решили сделать это видео публичным и искабельным. Можно смотреть с удовольствием :)

Привет. @Podlodka добавьте пожалуйста в описание видео ссылки, которые ребята обещали пошарить.

Все время проговорили про дизайн ревью, а как его правильно делать - так и не рассказали.

Господи, как же стыдно было слушать. Ни одно утверждение этих дяденек не выдерживает критики.

@zhuharev

Жыл бұрын

так распиши в чём проблема)

Бодро начали, а потом много воды. Не раскрыты практические темы: Ознакомление новичка с устоявшимися конвенциями на проекте. А они вообще поддерживаются? Или вы используете XP (а может дока дешевле и более унифицирована?) и обратное делегирование вас не пугает? Регламент code review. Тайминг: ожидания ответа, подтверждений, conversation chains, смена занятого или не отвечающего ревьювера. Критерии выноса больших коррекций в задачи рефакторинга или техдолг. Критерии декомпозиции уже существующей задачи. Декомпозиция при вскрытии скрытых, дополнительных проблем, новой информации. Снижение стоимости review за счет грамотной декомпозиции. Как и для чего вы используете митинг ретроспективы (если конечно он у вас есть)? Интеграция сборочных плагинов с CI/CD pipelines не пропускающие код без порога покрытия тестами разных типов. Или детский сад с забастовками у вас норма? Кто убирает препятствия? Особенно блокеры связанные с наделением доступа, отсутствия или неактуальность доки, кода, взаимозависимостями, неадекватные estimations, недостаточное описание требований к задаче и приемки, вскрывшимися новыми препятствиями. Скрам мастер, бизнес аналитик, или разработчик сам варится в длительных коммуникациях и у вам больно признать, что это не Scrum? Необходимое и достатоные acceptance criterias, прописание в задаче и следование всех ревьюверов этому при approve. Анализ по кайдзен таймингу самых длинных задержек approve'а и принятие адекватных приемов для удешевленя. Бесконечные цепочки изменений: правка рекомендаций вскрывает новые недостатки, которые затрагивают другой код, не относящийся к задаче, к которому появляется еще больше претензий. Отключение чек листов и регламента при правке критических продакшн багов или сходных задач, в которых важен фикс или code delivery в узких рамках времени. Конфликт агрессивного deadline с длительностью code review. Компромиссное качество. Достижение компромисса.