O
Вот. Мне кажется, что таких мержей будет много. Больше одного по крайней мере, а значит много ревью.
Size: a a a
2021 April 06
ЕГ
Ну смотрите. Либо вы делаете меньше МРов, в которых будет больше кода, либо вы делаете больше МРов, в которых будет меньше кода
ЕГ
Мне нравится второй вариант больше. Но варианта Меньше МРов и меньше кода не бывает)
O
Да, но гит заставляет меня привязывать ревью к мержу. А я считаю, что это нелогично и хочу привязать мерж к фиче.
ЕГ
Это вариант меньше мержей и больше кода в них
O
А можно сделать мерж, а потом в старый мерж реквест добавить новый код?
EB
А что ещё вам хочется? Команда разделяет вашу точку зрения? :)
O
Да, сейчас у нас так сделано.
ЕГ
Если его не закрыли, то да, запушенный код в ветку, из которой идёт мёрж, автоматом в него попадает. По крайней мерег Гитлаб и Битбакет так делает, практически уверен, что все остальные гит-системы так же
O
Я бы хотел перейти на гит, что бы иметь возможность делать ревью в идее. Но основное требование - фильтрация кода по человеку
O
А мерж при этом мог быть сделан?
EB
Кто-нибудь понял что это значит?
ЕГ
Мерж закрыт = мерж сделан, то есть код вмержен из одной ветки в другую уже. Какой смысл добавлять в него же ещё код, если он уже проведён?
O
Что б он попал в ревью
ЕГ
Сделать ещё один мерж и сделать ревью на него?)
EB
Ревью делается до мерджа. Потому что не надо лить говно в общие ветки
ЕГ
Я думаю там имелось ввиду "хочу привязать ревью к фиче") Но причин этого я тоже не очень понял
O
Если вот так получилось, что мне очень надо поделиться своим незаконченным кодом с другим разработчиком, мне надо придется мерж
O
это плохо и редко бывает, но бывает
НБ
ветку создать слабо?