Коллеги, сори за возможно тупой вопрос, получил пулл-реквест вот с

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

10 ответов

33 просмотра

Сделай split view

Павел-Алтынников Автор вопроса
Danya🔥
Сделай split view

нашёл как сделать, но к сожалению суть не поменялась(

Павел-Алтынников Автор вопроса
Danya🔥
Ну так хотя бы понятнее, нет?

нет) Тут смысл в том, что удалённый и добавленный код никак друг с другом не соотносятся. Если бы в данном методе удалили одну строку и одну добавили, то было бы ок.

Павел-Алтынников Автор вопроса

Что я тут хотел бы видеть: старый метод полностью красный, а новый метод полностью зелёный.

Павел Алтынников
Что я тут хотел бы видеть: старый метод полностью ...

Гит так диффы делает, увы, из-за того, что часть строк и скобок «переиспользована»

Павел-Алтынников Автор вопроса
Serge L.
Гит так диффы делает, увы, из-за того, что часть с...

Да, тоже на скобки подумал. Может есть какие-то хитрости? Пока на ум приходит сначала одним коммитом добавлять методы, а другим удалять.

Павел Алтынников
Да, тоже на скобки подумал. Может есть какие-то хи...

Это будет все звучать как костыли, к сожалению. Но вообще можно было новый метод создать в другом месте, или вот разными коммитами и ревьювить покоммитно, или вообще разнести на разные МРы

Похожие вопросы

Обсуждают сегодня

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
Magic
6
сделал сайт, прикрутил в боте сайт, и виджет логина. как автоматически логинить пользователя в аккаунт(телеграм), при входе с бота?
Александра Чернивецкая
5
Объясните, пожалуйста, почему компилятор ругается на использование в условии неинициализированной переменной: int x; Task.Run(async () => { x = await somefunc(); }).Wait...
Александр
5
Ребят, подскажите, пожалуйста, почему в префиксе к ассетам, которые генерируются через фильтр | theme в шаблоне, стал вдруг появляться index.php? Вот так выглядит ссылка на а...
Виталий
1
Всем привет. Ребята, подскажите, пожалуйста. у ботов есть ограничение на отправку сообщений - 30 сообщений в секунду, эти ограничения накладываются на все сообщения? или на со...
Artem Stormageddon
4
Блин, ребята, сори за тупые вопросы. А можно ли как-то открыть вебапку по нажатию на кнопку в меню(которое появляется слева, команды)?
Artem Stormageddon
3
а плаксы из-под питона умеют только в комфортных условиях что-то выдавить из себя?)
Lencore
9
Но, может, есть уже проверенная? Наши требования такие: 1. Сообщения должны приходить из Инста в CRM оду 2. Должна быть возможность подключить несколько экаунтов Инстаграм. Р...
Alexander Sharoiko MSE / Александр Шаройко
13
Это может быть все-таки не флудвейт? у меня ботфазер принимает изменения и отображает даже что они изменились, на видео видно что он прислал якобы уже измененное описание, н...
OVERLINK
13
Коллеги, может знает кто, можно ли цвет бейджа счётчика в BackendMenu менять без бубнов?
Alex Blaze
3
Карта сайта