Ок. Через пол часа гляну

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

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

так как ваш МР в остальном оч хорош :), то я приму его, но в будущем прошу так не делать

@terrakok Если не ошибаюсь, вы мне сказали избавиться от конфликтов ( = вмержить дев), а с веткой фичи я ничего не могу делать у меня же нет доступа к проекту

Валик, Костя правильно заметил, что нужно не мержить, а ребейзить.

я точно такую же ошибку допустил в mr cо списками файлов.

таким образом получается, что ветка как будто "начата" от свежего девелопа, правильно? В своих проектах тоже встречаюсь с этим на ревью, но даже мысли не было ребейс сделать) Вы всегда этим приёмом пользуетесь?

@CraggyHaggy спасибо за замечание, я наконец допер 🙈

за что мне, все лавры Косте

я вот сам допер, когда его сообщение прочитал.

ну если бы не ты, я бы не понял всё что хотел сказать @terrakok

главное прошу: аккуратнее с ребейзом! можно все сломать)
только что, минуту назад, разбирались с одним коллегой, что пошло не так.

правило: при разрешении конфликтов ребейза надо менять код ТОЛЬКО внутри конфликтов! никогда не трогайте что-то рядом!

ато будут дублирующиеся коммиты и другое зло

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

если мержить девелоп в фичу, а потом обратно, то в списке изменений этой фичи будут повторяться изменения девелопа, и ревью проводить неудобно. Или есть другой способ этого избежать?

и что с того? Ну по коммитам не оч удобно будет смотреть да, смотреть полный дифф с девом

всем надо уметь пользоваться.

Астрологи объявили неделю простреленных ног

😂😂😂