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

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

10 ответов

31 просмотр

Сделай split view

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

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

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

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

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

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

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

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

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

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

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

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

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

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

Ребята, всем привет. Подскажите, пожалуйста, можно ли как-то через бота понять, что этого бота добавили в группу\канал и выдали ему права администратора?
Artem Stormageddon
9
Это переведённый текст с английского. Я не говорю на русском, но могу использовать переводчик Телеграм. Приветствую! Я начинающий веб-разработчик и все еще учусь. В настояще...
𐩱𐩪𐩣𐩱𐩲𐩺𐩡
3
А не хотим ли мы развлечься? 😉 Но так чтобы с пользой для наших профессиональных навыков?? 👨‍🎓👩‍🎓 Предлагаю на октябрь запланировать тестовый запуск новой командной игры "Игр...
Andrii Kurdiumov
2
Привет всем! Почему этот код не срабатывает при добавлении или удалении пользователя из чата? bot.on('chat_member', async (ctx) => { console.log(ctx); }) bot.launch({allo...
Alexander
5
у кого сколько оперативы на базе данных ?
АДИЛЬБЕК
4
Через бот апи возможно получить ID стикерпака? Не ссылку.
Vexylon [АФК до 09.09]
5
Привет Хочу сделать аналог iCloud’а для своих проектов, чтобы пользовательская информация хранилась в облаке, была доступна во всех сервисах, её можно было подсасывать везде)...
Виталий
9
В тг можно спарсить всех кто пишет в группе? Если список участников скрыт
S
3
код Event::listen('cms.page.display', function (&$content, $slug, $page, $html) { if (is_object($content)) { dump($content); } else { dump($s...
Point 111
3
Всем привет. Не понимаю, в чём тут шутка юмора. Убирается только разрешение на send_messages. А send_media_messages остаётся. Как сделать, чтобы оба убирались? await b...
Alexander
2
Карта сайта