Size: a a a

2021 April 21

EO

Eugene Obrezkov in Frontend UA
и на таких проектах мы сквошим и только сквошим, потому что нет никаких гарантий что в том ПР хорошая история
источник

AR

Alexey Raspopov in Frontend UA
я к тому и веду, кому вообще нужна эта история в пулл реквестах
источник

VS

V7v S6k in Frontend UA
а кому потрібні коміти з 30 файлів в мастері?
источник

AR

Alexey Raspopov in Frontend UA
пулл реквест как коммит это прям идеально: точка в истории, прямая ссылка на контекст, название пулл реквеста good enough как заголовок коммита
источник

AR

Alexey Raspopov in Frontend UA
за такие пулл реквесты просто руки отрывать нужно
источник

VS

V7v S6k in Frontend UA
особливо якщо там ще рефакторинг, ага
источник

VS

V7v S6k in Frontend UA
з перейменуванням файлів
источник

VS

V7v S6k in Frontend UA
ні, ну якщо в тебе пр з одного атомарного коміту, то ок
источник

AR

Alexey Raspopov in Frontend UA
ну не надо ж прям в крайности
источник

VS

V7v S6k in Frontend UA
але типу за кожен коміт бігати відкривати пр теж таке
источник

AR

Alexey Raspopov in Frontend UA
этого я тоже не предлагаю
источник

EO

Eugene Obrezkov in Frontend UA
если ты сидишь такой работаешь над фичей Х, а потом оп - блокер - надо починить что-то в другом месте, чтобы мог продолжить

в таких случаях бывают частые 2 варианта:

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

в случае с вариантом №2, несмотря на сквоши, вся история всё равно сохраняется минимальными усилиями (не нужно историю переставлять местами)
проблема варинта №2, что даже я бывает грешу вариантом №1 и тут уже вся надежда на коллективное ревью
источник

EO

Eugene Obrezkov in Frontend UA
зачастую мы такие ПР режектим и просим вынести части в отдельные ПР, которые можно протестировать в отрыве от фичи Х
источник

EO

Eugene Obrezkov in Frontend UA
чтобы хотя бы понимать, что это ничего не сломало
а история этого всего очень даже неплохо хранится, даже со сквошами
источник

EO

Eugene Obrezkov in Frontend UA
тебе ещё и ссылка на ПР будет, в котором вся история
источник

EO

Eugene Obrezkov in Frontend UA
сквош коммиты в мастер из ПР это как ещё одни бакеты поверх коммитов

вот есть фича Х - она в этом коммите
хочешь посмотреть историю - вот ПР в котором она пилилась со всеми коммитами из ветки
источник

EO

Eugene Obrezkov in Frontend UA
если, конечно, автор ПР не балуется ребейсом с форсом
источник

SG

Stas G in Frontend UA
Лёша, ты работаешь над вашим основным продуктом или над внутренними тулами?
источник

SG

Stas G in Frontend UA
Я просто не представляю, как в продукте можно контролировать размер МРа)
источник

AR

Alexey Raspopov in Frontend UA
работаю над продуктом, но не на основном монолите
источник