Маленькие или большие PR?

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

В остальном маленькие PR – это проблема. Вы размазываете разработку по большому количеству небольших изменений и во время ревью теряете контекст. 

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

Если вы во время ревью следите только за именами переменных и придираетесь к синтаксису – то да, вам нужны небольшие изменения. Если за этим следит ИИ, то вам плевать на мелочи, вы смотрите на картину в целом, на архитектуру, на решение, а не на переменные. 

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

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

Маленькие PR имеют место быть, но не стоит боятся больших PR и искусственно или насильно разбивать решение и размазывать контекст.

А какая разница - 1 PR на 1000 строк или 10 по 100 строк? Тебе всё равно придётся ревьюить 1000 строк? Разница в том, что в первом случае вы будете отвлекаться один раз, чтобы посмотреть 1000 строк, а во втором 10 раз. Первое займёт у вас меньше времени. 

P.S. Не знаю зачем, но ИИшка создала у этой ящерицы три руки. Причём две руки на стороне маленьких PR, видимо потому что там нужно больше руками работать, в при больших PR больше головой. 

---

Понравилось? Кликни Лайк, чтобы я знал, какой контент более интересен читателям. Заметку уже лайкнули 1 человек