Назвал ветку не так и коммит, запушил и сделал PR.

Мне просто обновить названия веток и коммитов локально и удаленно и все?

13 ответов

13 просмотров

Коммиты можно поменять. А вот поменять исходную ветку в МР обычно на хабах нельзя.

Kirill-Kulikov Автор вопроса
Serge L.
Коммиты можно поменять. А вот поменять исходную ве...

а удалить PR можно (его вряд ли сегодня вообще будет смотреть)? я же могу потом переназвать ветку и сделать новый PR?

Kirill Kulikov
а удалить PR можно (его вряд ли сегодня вообще буд...

Обычно удалять нельзя, только закрывать. Но настолько ли критично, что вы ветку назвали как-то не так?

Kirill-Kulikov Автор вопроса
Serge L.
Обычно удалять нельзя, только закрывать. Но настол...

ну т.е. я один этот вопрос не решу уже да? Что можно максимум сделать?))

Kirill-Kulikov Автор вопроса
Kirill Kulikov
ну т.е. я один этот вопрос не решу уже да? Что мож...

например если я сам отклоню этот PR, удалю ветку, залью новую и создам новый PR. Вариант же, да?

Kirill-Kulikov Автор вопроса
Alexander Epaneshnikov
да. но я бы за такое не погладил по голове.

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

Kirill Kulikov
ну мне наоборот нужен способ наиболее адекватный. ...

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

Kirill Kulikov
Хорошо, спасибо. так и сделаю

если вас будут бить за это - скажите что саня из чата так сказал. 🙂

Kirill Kulikov
Хорошо, спасибо. так и сделаю

Так а скажите, что не так с названием ветки? Интересно же 😸

Serge L.
Так а скажите, что не так с названием ветки? Интер...

я так понял - оно должно было bug/xxx а создал feature/xxx

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

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

Добрый вечер, Пока не совсем понимаю как наладить общение между телеграм ботом и ПО для работы с сим боксом. По самому боту так понял: - Нужен некий баланс, который можно поп...
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
Карта сайта